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

Sheen not working in combination with hdr env? #22359

Closed
vector3 opened this issue Aug 18, 2021 · 4 comments · Fixed by #23069
Closed

Sheen not working in combination with hdr env? #22359

vector3 opened this issue Aug 18, 2021 · 4 comments · Fixed by #23069

Comments

@vector3
Copy link

vector3 commented Aug 18, 2021

Describe the bug
Sheen rendering doesn't seem to work in combination with an HDR environment map.

To Reproduce

Steps to reproduce the behavior:

  1. Load an HDR environment map
  2. Set the scene's environment property to that map
  3. Add some generic mesh (e.g. sphere) to the scene
  4. Create a MeshPhysicsMaterial and set its sheen property to some color value.
  5. Assign the material to the mesh.

Expected behavior

The sheen should render even when using an HDR environment map, Instead, you have to set the envMapIntensity property of the material to a very small value or zero to be able to see the sheen at all.

Platform:

  • Device: Desktop
  • OS: Windows
  • Browser: Chrome
  • Three.js version: 131
@Mugen87
Copy link
Collaborator

Mugen87 commented Aug 18, 2021

Related #17303.

@elalish
Copy link
Contributor

elalish commented Sep 1, 2021

@elalish
Copy link
Contributor

elalish commented Sep 1, 2021

@WestLangley Any interest in trying your hand at this? I'm kicking myself I haven't found time for this yet, but I may have to this release cycle if no one else does...

@SixBytesUnder
Copy link

HI @elalish
Did you have a chance to look into this one?
I'd love to know if you have any idea of a timeline to fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants