Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error if SceneReader is given an invalid path #528

Closed
bentoogood opened this issue Sep 11, 2013 · 0 comments · Fixed by #570
Closed

Error if SceneReader is given an invalid path #528

bentoogood opened this issue Sep 11, 2013 · 0 comments · Fixed by #570
Labels
bug Issues representing bugs core Issues with core Gaffer functionality

Comments

@bentoogood
Copy link
Contributor

To reproduce - create scene reader, enter a path to a non-existent scene cache.

ERROR : SceneProcedural::render() : /asdasda/sdfsdf.scc
ERROR : EventLoop.__qtIdleCallback : Traceback (most recent call last):
  File "/software/apps/gaffer/0.78.0/cent6.x86_64/cortex/8/gaffer/python/GafferUI/EventLoop.py", line 241, in __qtIdleCallback
    if not c() :
  File "/software/apps/gaffer/0.78.0/cent6.x86_64/cortex/8/gaffer/python/GafferUI/PathListingWidget.py", line 404, in __pathChangedUpdate
    self.__update()
  File "/software/apps/gaffer/0.78.0/cent6.x86_64/cortex/8/gaffer/python/GafferUI/PathListingWidget.py", line 325, in __update
    dirPath = self.__dirPath()
  File "/software/apps/gaffer/0.78.0/cent6.x86_64/cortex/8/gaffer/python/GafferUI/PathListingWidget.py", line 367, in __dirPath
    del pp[-1]
  File "/software/apps/gaffer/0.78.0/cent6.x86_64/cortex/8/gaffer/python/Gaffer/Path.py", line 269, in __delitem__
    self.__items.__delitem__( index )
IndexError: list assignment index out of range
johnhaddon added a commit to johnhaddon/gaffer that referenced this issue Oct 2, 2013
@johnhaddon
Fixed PathListingWidget errors when "/" path is invalid.
99c9feb 
Invalid root paths occur quite frequently in the SceneHierarchy editor when an invalid filename has been entered in a SceneReader or AlembicReader. Those nodes will still display an error, but the SceneHierarchy now won't output a confusing stacktrace which distracts people from the root cause.

Fixes GafferHQ#528.
@johnhaddon johnhaddon mentioned this issue Oct 2, 2013
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issues representing bugs core Issues with core Gaffer functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed PathListingWidget errors when "/" path is invalid. johnhaddon/gaffer
1 participant
@bentoogood