ynput / ayon-houdini

Houdini addon for AYON
Apache License 2.0
11 stars 9 forks source link

Publishing usdrender without LOP path set raises confusing error on collect #71

Closed BigRoy closed 2 months ago

BigRoy commented 3 months ago

Is there an existing issue for this?

Current Behavior:

When the LOP path on the USD Render ROP node remains empty the publishing results in a confusing error:

Traceback (most recent call last):
  File "C:\Users\User\AppData\Local\Ynput\AYON\dependency_packages\ayon_2406251801_windows.zip\dependencies\pyblish\plugin.py", line 528, in __explicit_process
    runner(*args)
  File "E:\dev\ayon-houdini\client\ayon_houdini\plugins\publish\collect_local_render_instances.py", line 65, in process
    expectedFiles = next(iter(instance.data["expectedFiles"]), {})
KeyError: 'expectedFiles'

Expected Behavior:

It should not error on collect and instead provide a nice validation report that the output node is not set or incorrectly configured (which I think may already be the case if the error on the collector itself is resolved.)

Version

1.0.0

What platform you are running on?

Windows

Steps To Reproduce:

  1. Create USD Render ROP (without setting a LOP path) and set to 'render local'
  2. Publish

Are there any labels you wish to add?

Relevant log output:

No response

Additional context:

No response