-
Notifications
You must be signed in to change notification settings - Fork 8
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.
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
Maya: Fix update reference without reference node showing wrong error message #3
Maya: Fix update reference without reference node showing wrong error message #3
Conversation
…a reference update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have followed the steps....as said, no any error raised via UI just the following log in the script editor:
# Traceback (most recent call last):
# File "C:\Work\REPO\ayon-core\client\ayon_core\tools\sceneinventory\view.py", line 366, in <lambda>
# lambda: self._show_version_dialog(item_ids, active_repre_id)
# File "C:\Work\REPO\ayon-core\client\ayon_core\tools\sceneinventory\view.py", line 843, in _show_version_dialog
# self._update_containers_to_version(
# File "C:\Work\REPO\ayon-core\client\ayon_core\tools\sceneinventory\view.py", line 1032, in _update_containers_to_version
# self._update_containers(item_ids, versions)
# File "C:\Work\REPO\ayon-core\client\ayon_core\tools\sceneinventory\view.py", line 1007, in _update_containers
# update_container(container, item_version)
# File "C:\Work\REPO\ayon-core\client\ayon_core\pipeline\load\utils.py", line 539, in update_container
# return Loader().update(container, context)
# File "c:\Work\REPO\ayon-maya\client\ayon_maya\plugins\load\load_reference.py", line 256, in update
# super(ReferenceLoader, self).update(container, context)
# File "c:\Work\REPO\ayon-maya\client\ayon_maya\api\plugin.py", line 808, in update
# raise LoadError("No reference node found in container")
# ayon_core.pipeline.load.utils.LoadError: No reference node found in container
Thanks @LiborBatek - the lack of UI dialog on the error is indeed an issue, but a separate one from reporting the correct error starting here. The UI issue would need changes in |
Handling of errors in scene inventory is a different beast. |
@BigRoy thx for clarification, as I wasnt sure if thats all needed for Approval review, call it my lack of knowledge :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works ok.
Note
PR moved from ynput/ayon-core#465
Changelog Description
Raise
LoadError
instead ofassert
on no reference node found on Maya reference updateAdditional info
Fixes #9
Note: This now currently shows no dialog.
Testing notes:
Easiest way to get into a 'broken' state for the ReferenceLoader:
AVALON_CONTAINERS
add an object into the reference container (just so that maya does not auto-delete it in next step)