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.

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

USD ROP: Add validator for Implicit Layer Save paths #90

Open
2 tasks done
BigRoy opened this issue Sep 5, 2024 · 0 comments
Open
2 tasks done

USD ROP: Add validator for Implicit Layer Save paths #90

BigRoy opened this issue Sep 5, 2024 · 0 comments
Assignees
Labels
type: enhancement Improvement of existing functionality or minor addition

Comments

@BigRoy
Copy link
Contributor

BigRoy commented Sep 5, 2024

Is there an existing issue for this?

  • I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?

Implicit layer save paths aren't 'collected' by the current publishing logic to be included - because for most studios adopting USD with Solaris the first time these so called "implicit layers" were not intentionally created to become their own USD layers, but just a byproduct of the default values of e.g. a SOP import.

In essence, they are not intentionally generated and should actually not have been separate layers.

How would you imagine the implementation of the feature?

Add a validator that recognizes all the implicit layers the USD ROP would write out and report them.

The validator could potentially even have e.g. repair actions like "Set explicit save path" or "Flatten layer" if we could somehow auto-detect where it is in the graph, etc. but that may be complicated with custom HDAs

Are there any labels you wish to add?

  • I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:

We could also just take along implicit layers for the ride in the publishing but 99% of the time they just aren't intentional.

Additional context:

The issue or confusion comes up every now and then, e.g. in this issue here

Also note that this would only be published currently if a checkbox is manually disabled on the USD ROP that Houdini has enabled by default:
image

@BigRoy BigRoy added the type: enhancement Improvement of existing functionality or minor addition label Sep 5, 2024
@BigRoy BigRoy self-assigned this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

No branches or pull requests

1 participant