-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Color Management changes in THREE #5302
Comments
Hi @dmarcos! This summary is worth a look, if you haven't read it yet: At a glance, it looks like the "environment" component might need some updates, as described in supermedium/aframe-environment-component#83. Are there other issues you're seeing outside the environment? If that's the main thing, I can take a closer look there... Setting |
Thanks for getting back so quickly. I just skimmed through the thread. Still so much for me to learn he he. I was writing a component and handling textures manually and learned I have to do: texture.colorSpace = THREE.SRGBColorSpace; We are doing that automatically already for the built-in materials For the environment component I have to spend more time. Thanks so much. |
@dmarcos While setting As for the |
I've seen changes landing on THREE lately related to color management that seemed change A-Frame default behavior
A-Frame 1.4.2 on THREE r147
A-Frame master on THREE r152
@donmccurdy Sorry to bother. I went through the changelog. There are tons of changes and couldn't make it complete sense. Do you have a quick summary of the changes we have to do to get the same results than r147? Thanks so much
The text was updated successfully, but these errors were encountered: