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

ComplianceChecker issues #1760

Open
dgovil opened this issue Feb 5, 2022 · 3 comments
Open

ComplianceChecker issues #1760

dgovil opened this issue Feb 5, 2022 · 3 comments

Comments

@dgovil
Copy link
Collaborator

dgovil commented Feb 5, 2022

I just wanted to report a few issues we hit in the complianceChecker code base with USD 21.11

  1. Some of the tests reference AddFailedCheck instead of _AddFailedCheck and fail due to the missing attribute
    https:/PixarAnimationStudios/USD/blob/faed18ce62c8736b02413635b584a2f637156bad/pxr/usd/usdUtils/complianceChecker.py#L417

  2. The UsdShade::ConnectableAPI fails when it encounters a prim type that ti doesn't know about. This causes it to fail running the tests rather than just being surfaced as an error when an unknown prim is encountered. This happens in the C++ layer and doesn't seem to get bubbled up properly.

#usda 1.0

def Foo "MyPrim" () {}
@spiffmon
Copy link
Member

spiffmon commented Feb 5, 2022

Thank you , @dgovil ! Problem #1 is fixed in 22.03. I remember also fixing a problem with the ConnectableAPIBehavior registry, but don't recall if that was it, specifically.

@dgovil
Copy link
Collaborator Author

dgovil commented Feb 5, 2022

Awesome, thanks, Spiff!

@jilliene
Copy link

jilliene commented Feb 8, 2022

Filed as internal issue #USD-7180

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants