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

Need strict definition of black border color in UsdPreviewSurface Texture Reader #2770

Closed
syoyo opened this issue Oct 30, 2023 · 1 comment
Closed

Comments

@syoyo
Copy link

syoyo commented Oct 30, 2023

Description of Issue

https://openusd.org/release/spec_usdpreviewsurface.html#version-2-5-current-head

The spec says

Wrap mode when reading this texture.Possible Values:

        black : Reader returns black outside unit square

But not sure what actual black color should be.

Possible value is either transparent black (0, 0, 0, 0) or opaque black (0, 0, 0, 1)

We need strict definition of black border color.
(NOTE: Currently TinyUSDZ Tydra uses transparent black)

I'm not sure how Storm interprets black border color, but it seems Storm uses TransparentBlack as a default border color: https://github.com/PixarAnimationStudios/OpenUSD/blob/0b18ad3f840c24eb25e16b795a5b0821cf05126e/pxr/imaging/hgi/sampler.h#L73C18-L73C18

cc @dgovil

Steps to Reproduce

N/A

System Information (OS, Hardware)

N/A

Package Versions

23.08

Build Flags

N/A

@sunyab
Copy link
Contributor

sunyab commented Oct 30, 2023

Filed as internal issue #USD-8890

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