-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Clips template asset path support sdf file format arguments #1450
Clips template asset path support sdf file format arguments #1450
Conversation
96065e2
to
4a07b7e
Compare
Filed as internal issue #USD-6571 |
Hello @ainaerco Thanks for filing the PR. The test you added is failing in all our internal testing platforms and configurations. Will it be possible for you to address the failing test?
Thanks |
4a07b7e
to
30ccd87
Compare
Sorry, I was testing in usd 19.11. |
Hello, is there anything else I can do to get this merged? |
Hi @ainaerco -- the test you added fails for us as @tallytalwar mentioned. Does it pass for you when using the dev branch? 19.11 is quite old and lots of things may have changed. |
I have pushed commit after it was reported that it was failing. Is it failing still? |
Hi @ainaerco Thanks for your patience. The test you are adding is still failing for us:
I also noticed that there is no difference between your previous commit: 4a07b7e and your forced push commit: 30ccd87. Can you please try running the test on your end and update this PR with a fix? |
30ccd87
to
63e9718
Compare
I think I fixed it now, thanks for checking |
Description of Change(s)
Clips template asset paths support sdf file format arguments.
Discussion is here:
https://groups.google.com/g/usd-interest/c/7SxzTgBVfv4
Fixes Issue(s)