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

MaterialX texcoord-related regression in v24.03 #3048

Closed
pablode opened this issue Apr 20, 2024 · 4 comments
Closed

MaterialX texcoord-related regression in v24.03 #3048

pablode opened this issue Apr 20, 2024 · 4 comments

Comments

@pablode
Copy link
Contributor

pablode commented Apr 20, 2024

Description of Issue

Following asset looks different after upgrading USD: Buffer_Interleaved_01.zip

v23.08
bug_2308

v24.03
bug_2403

(see comment below for an explanation)

Steps to Reproduce

  1. open the asset in usdview

System Information (OS, Hardware)

Occurs on both Windows (OpenGL) and macOS (Metal)

Package Versions

v24.03 and v24.05 are affected

Build Flags

--openimageio --opencolorio

@pablode pablode closed this as completed Apr 20, 2024
@pablode
Copy link
Contributor Author

pablode commented Apr 20, 2024

I believe to have found the cause. First, the texture coordinates are faulty:

texCoord2f[] primvars:st0 = [(0, 1), (1, 0), (5.831554e-39, 1), (0, 1), (5.831554e-39, 1), (0, 1)] (
    interpolation = "vertex"
)

However, they are irrelevant because the BaseColor_Grey.png texture that is being sampled is 100% grey and is defined with the periodic address mode in both u and v directions. Hence, regardless of UV values, grey should be returned and not black.

Although on macOS the color is solid black, I have observed flickering of the correct colors with the OpenGL backend. Perhaps this is somehow related to the newly-introduced tangent frame computation?

@pablode pablode reopened this Apr 20, 2024
@pablode pablode changed the title MaterialX geomprop reading regression in v24.03 MaterialX texcoord-related regression in v24.03 Apr 21, 2024
@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-9565

@pablode
Copy link
Contributor Author

pablode commented Jul 14, 2024

On macOS, I can't reproduce this anymore with usdview, but it's 100% reproducible with usdrecord.
usdrecord Buffer_Interleaved_01/Buffer_Interleaved_01.usda bug.png to get above render.

@pablode
Copy link
Contributor Author

pablode commented Dec 30, 2024

Can't reproduce with USD v24.11.

@pablode pablode closed this as completed Dec 30, 2024
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

2 participants