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

Hydra scene index mode doesn't resolve UDIM paths #3492

Open
marktucker opened this issue Jan 23, 2025 · 5 comments
Open

Hydra scene index mode doesn't resolve UDIM paths #3492

marktucker opened this issue Jan 23, 2025 · 5 comments

Comments

@marktucker
Copy link
Contributor

Try to render the attached scene with USDIMAGINGGL_ENGINE_ENABLE_SCENE_INDEX=1.
If you start the render process from some directory other than the one containing the USDA file, the UDIM maps will not be found (if the cwd for the process is where the USDA file is, chances are the renderer will find the UDIMs using the unresolved relative paths).

I can see no evidence that when using scene index plugins that UDIM paths are run through UsdShadeUdimUtils::ResolveUdimPath as they are when using the legacy hydra delegate.

udim.zip

@marktucker
Copy link
Contributor Author

If this is not currently on your radar to be fixed for 25.05, I can try to come up with a PR.

@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-10592

(This is an automated message. See here for more information.)

@tcauchois
Copy link
Contributor

This is the first I've seen this, although it doesn't look too hard to fix. I'll gladly accept any PRs though!

We could consider specializing UsdImagingDataSourceAttribute to check for the "" string and call this; or have UsdImagingDataSourceMaterial wrap asset parameters and call this... While I don't particularly expect UDIMs outside of shader inputs, there's something nice about handling it uniformly in our baseline asset path handling. Let me know what you think!

Regardless, we'll try to address this for 25.05. It seems relatively high value for effort.

@marktucker
Copy link
Contributor Author

Just adding a comment to make sure you see that I did submit a PR for this...

@tcauchois
Copy link
Contributor

Ahh thanks, missed that!

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