Skip to content

ci-vfxall: rely on OpenUSD MaterialX resources - #453

Merged
jfpanisset merged 1 commit into
AcademySoftwareFoundation:mainfrom
nicolaspopravka:fix/remove-openusd-materialx-workaround
Aug 2, 2026
Merged

ci-vfxall: rely on OpenUSD MaterialX resources#453
jfpanisset merged 1 commit into
AcademySoftwareFoundation:mainfrom
nicolaspopravka:fix/remove-openusd-materialx-workaround

Conversation

@nicolaspopravka

@nicolaspopravka nicolaspopravka commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Remove the global PXR_MTLX_STDLIB_SEARCH_PATHS workaround from ci-vfxall.

OpenUSD 26.03 and newer include a fix added by PR #2904 (PixarAnimationStudios/OpenUSD#2904). It updates usdMtlx’s CMakeLists.txt so the MaterialX standard library is installed with the usdMtlx plugin and can be found without the environment variable. I think it is ok for the issue to remain in the older CYs instead of bringing the workaround into newer CYs.

Testing confirmed:

  • CY2023–2025 require the workaround.
  • CY2026–2027 work without it.

Test results: https://github.com/nicolaspopravka/aswf-docker/actions/runs/30363851174

Follow-up to #452; revisits #451.

Assisted-by: OpenAI Codex / GPT-5

Signed-off-by: Nicolas Popravka <nicolaspopravka@gmail.com>
@nicolaspopravka
nicolaspopravka marked this pull request as ready for review July 28, 2026 14:22

@jfpanisset jfpanisset left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OpenUSD 26.03 went into the images:

ci-2026-clang19.4
ci-2026-clang20.4

The images:

ci-2027-clang21.0
ci-2027-clang22.0

have 26.05, although for the "real" release of 2027 images I'm thinking of going to 26.08.

Do you know which version of ci-vfxall you were testing against where you were seeing MaterialX related failures? And are you OK with MaterialX in OpenUSD support only working with current 2026 / 2027 ci-vfxall images?

@nicolaspopravka

Copy link
Copy Markdown
Contributor Author

Thanks. Yes, I’m ok with MtlX support without the environment workaround only in CY2026/CY2027 images, and with leaving the older CY images as-is.

I tested the aswf/ci-vfxall images for all CYs, 2023 to 2027. OpenChessSet renders with fallback/grey materials in each CY, so unfortunately I see a MaterialX rendering failure across the entire CY range.

The exact failures differ : CY2026 and CY2027 explicitly report the missing libraries/stdlib/genglsl/lib/mx_math.glsl include; earlier years render the grey-shaded image even if the render command returning success.

The specific issue reported in #454 is the CY2027 run using aswf/ci-vfxall:2027.

Moving the "real" CY2027 images to OpenUSD 26.08 sounds good. Happy to rerun the same benchmark scene against that image when it's available.

@jfpanisset
jfpanisset merged commit f763c16 into AcademySoftwareFoundation:main Aug 2, 2026
2 of 3 checks passed
@nicolaspopravka
nicolaspopravka deleted the fix/remove-openusd-materialx-workaround branch August 7, 2026 09:25
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

Successfully merging this pull request may close these issues.

2 participants