usdValidation: usdGeom: YUpAxisValidator

[beersandrew]

Description of Change(s)

• Moving the existing portion of the StageMetadataChecker from complianceChecker.py that verifies the upAxis is Y
• Added implementation code, a test, and necessary tokens & plugInfo entries

Link to proposal (if applicable)

Fixes Issue(s)

• https://github.com/orgs/PixarAnimationStudios/projects/3/views/1?pane=issue&itemId=65424791

Note: This PR is a replacement for #3399

Checklist

[X] I have created this PR based on the dev branch

[X] I have followed the coding conventions

[X] I have added unit tests that exercise this functionality (Reference:
testing guidelines)

[X] I have verified that all unit tests pass with the proposed changes

[X] I have submitted a signed Contributor License Agreement (Reference:
Contributor License Agreement instructions)

[jesschimein]

Filed as internal issue #USD-10491

[jesschimein]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[tallytalwar]

Instead of doing a Up axis being Y check, I think this should be a stageMetadataValidator (https://github.com/PixarAnimationStudios/OpenUSD/blob/dev/pxr/usdValidation/usdValidation/coreValidators.cpp) which checks if a upAxis is authored or not and if not error on it.

As for up axis being "y" or not shouldn't be a validator provided with openUSD. It seems from current compliance checker code that thats a arkit/scenekit requirement, and hence should also be part of suite of validators included in scenekit? cc. @dgovil

[beersandrew]

Instead of doing a Up axis being Y check, I think this should be a stageMetadataValidator (https://github.com/PixarAnimationStudios/OpenUSD/blob/dev/pxr/usdValidation/usdValidation/coreValidators.cpp) which checks if a upAxis is authored or not and if not error on it.

This one is already covered here. Can this PR be closed?

[tallytalwar]

Yup I think so. Thanks @beersandrew