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

Bring MaterialX stdlib search paths in HdStorm on par with the ones used in MtlxPlugin #1628

Closed

Conversation

vlasovi
Copy link
Contributor

@vlasovi vlasovi commented Sep 20, 2021

Description of Change(s)

HdStorm now uses environment variable "PXR_USDMTLX_STDLIB_SEARCH_PATHS" to set additional MaterialX stdlib search paths, which fully mimics the behaviour of MtlxPlugin.

Fixes Issue(s)

Partially addresses #1586 (MaterialX search paths are unstable not relocatable and code gen breaks when relocating libraries)

@jilliene
Copy link

Filed as internal issue #USD-6909

@klucknav
Copy link
Contributor

klucknav commented Nov 4, 2021

Thank you for the change. Unfortunately we cannot directly merge your change as is, there are a handful of changes we would like to make. But we have a fix for the underlying issue based on your change that should be in dev soon. Thanks again.

@klucknav klucknav closed this Nov 4, 2021
@vlasovi vlasovi deleted the ogs/mtlx_search_path branch November 11, 2021 16:13
pixar-oss pushed a commit that referenced this pull request Nov 16, 2021
UsdMtlx uses the two environment variables PXR_USDMTLX_STDLIB_SEARCH_PATHS and
PXR_USDMTLX_PUGIN_SEARCH_PATHS for the user to specify where to find the necessary
MaterialX files. The imaging side does not do this. This change adds that behavior to HdMtlx.
So now, if the location of the MaterialX provided libraries folder changes from the location
originally found at runtime (stored in PXR_MATERIALX_STDLIB_DIR) the new location(s)
can be indicated with PXR_MTLX_STDLIB_SEARCH_PATHS. The location(s) to any custom
MaterialX files can be indicated with PXR_MTLX_PLUGIN_SEARCH_PATHS.

(Based on GitHub PR #1628 by @vlasovi)

Fixes #1586

(Internal change: 2197997)
lkerley pushed a commit to imageworks/USD that referenced this pull request Jan 7, 2022
UsdMtlx uses the two environment variables PXR_USDMTLX_STDLIB_SEARCH_PATHS and
PXR_USDMTLX_PUGIN_SEARCH_PATHS for the user to specify where to find the necessary
MaterialX files. The imaging side does not do this. This change adds that behavior to HdMtlx.
So now, if the location of the MaterialX provided libraries folder changes from the location
originally found at runtime (stored in PXR_MATERIALX_STDLIB_DIR) the new location(s)
can be indicated with PXR_MTLX_STDLIB_SEARCH_PATHS. The location(s) to any custom
MaterialX files can be indicated with PXR_MTLX_PLUGIN_SEARCH_PATHS.

(Based on GitHub PR PixarAnimationStudios#1628 by @vlasovi)

Fixes PixarAnimationStudios#1586

(Internal change: 2197997)
seando-adsk pushed a commit to autodesk-forks/USD that referenced this pull request Mar 1, 2024
UsdMtlx uses the two environment variables PXR_USDMTLX_STDLIB_SEARCH_PATHS and
PXR_USDMTLX_PUGIN_SEARCH_PATHS for the user to specify where to find the necessary
MaterialX files. The imaging side does not do this. This change adds that behavior to HdMtlx.
So now, if the location of the MaterialX provided libraries folder changes from the location
originally found at runtime (stored in PXR_MATERIALX_STDLIB_DIR) the new location(s)
can be indicated with PXR_MTLX_STDLIB_SEARCH_PATHS. The location(s) to any custom
MaterialX files can be indicated with PXR_MTLX_PLUGIN_SEARCH_PATHS.

(Based on GitHub PR PixarAnimationStudios#1628 by @vlasovi)

Fixes PixarAnimationStudios#1586

(Internal change: 2197997)
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.

3 participants