search

PixarAnimationStudios / OpenUSD

Universal Scene Description
http://www.openusd.org
Other
6.08k stars 1.21k forks source link

Add unique identifiers to `UsdValidatorError` #3241

Open nvmkuruc opened 2 months ago

nvmkuruc commented 2 months ago

Description of Issue

See the discussion here for more information. https://forum.aousd.org/t/identifiers-for-validation-errors

As one validator may generate multiple distinct error kinds (for example, the usdGeom stage metadata checker validates both the upAxis and metersPerUnit field), the request is that errors get an identifier or name field that can uniquely group or categorize them for downstream post processing, filtering, and organization without relying on error prone matching of the error message string.

Steps to Reproduce

System Information (OS, Hardware)

Package Versions

Build Flags

jesschimein commented 2 months ago

Filed as internal issue #USD-9999

nvmkuruc commented 4 weeks ago

@jesschimein @tallytalwar I think we can close this?

tallytalwar commented 4 weeks ago

I think once the change is in release branch it should close automatically. But yes can be closed. Thanks for being on top of things @nvmkuruc