search

ynput / ayon-houdini

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

Remove the :SDF_FORMAT_ARGS: from the collected filenames for Explicit Layer Save paths #159

Closed BigRoy closed 3 weeks ago

BigRoy commented 3 weeks ago

Changelog Description

Remove the :SDF_FORMAT_ARGS: from the collected filenames for Explicit Layer Save paths

Additional review information

This may fix the issue mentioned here: https://community.ynput.io/t/houdini-usd-publish-errors/1944?u=bigroy

Testing notes:

  1. Create asset with component builder
  2. Publish as USD
BigRoy commented 3 weeks ago

Do we need to have usdAsset file that sublayers the asset created by the component builder? is it beneficial for having a consistent contribution workflow? (what about the default prim validator. can we bypass it?)

In this case it's more of a bugfix than a recommendation on the workflow. Preferably the user is capable of publishing this WITHOUT the need of the extra USD layer the USD ROP writes - and instead is capable of just publishing the Component Builder output directly (which would be the ROP inside the component builder.)

However, that'd be a separate issue to investigate.

Also, what about having one more preset (use default prim from the asset) ?

That wouldn't be valid with the USD contribution workflow unfortunately - since it just requires the default prim to be your current folder name. Not much freedom there.

MustafaJafar commented 3 weeks ago

Also, loading works. I'm not how to test the re-pathing thing in https://github.com/ynput/ayon-houdini/pull/159/commits/d504edb3ea4352e507eddff64eefbd296bc52e27. Animation_29

BigRoy commented 3 weeks ago

Also, loading works. I'm not how to test the re-pathing thing in d504edb.

In the Houdini Solaris > Scene Graph Layers panel you should be able to investigate the actual layers it loaded for that stage. The layers from the published file should refer to the published paths, not the paths in your work area. That's it.