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

Tweak environment tonemapping and adjustments for Godot 3.4 #106

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Sep 13, 2021

  • Use the new ACES Fitted tonemapping for physically correct behavior of glowing lights.
  • Decrease overall saturation and contrast for a less jarring appearance.

Additionally, I've thought about adding subtle ambient lighting to prevent some areas from being too dark, but this is likely better done in a separate PR. We could also increase some lights' ranges and tweak their attenuation, but this may impact performance negatively as lights may impact more meshes.

Note: Don't merge before Godot 3.4 is released to avoid breaking the scene's appearance in 3.3.x.

Preview

Before After
Before After

- Use the new ACES Fitted tonemapping for physically correct behavior
  of glowing lights.
- Decrease overall saturation and contrast for a less jarring appearance.
@Calinou Calinou marked this pull request as draft September 13, 2021 17:36
@aaronfranke
Copy link
Member

aaronfranke commented Sep 13, 2021

Lowering the contrast and whitening bright lights is a subjective change, so I'd like to get feedback on this first.

It may be better to instead/also lower the brightness of the blue lights, since AFAIK the deepish blue was the artistic intent.

@Calinou
Copy link
Member Author

Calinou commented Sep 13, 2021

It may be better to instead/also lower the brightness of the blue lights, since AFAIK the deepish blue was the artistic intent.

We could ask Fernando (the demo's artist) on Twitter about this 🙂 Does he have a GitHub account to comment on this issue?

@Lauson1ex
Copy link

I like it! Looks very cinematic. I'm also all in for changing the attenuation curve of lights.

@Calinou
Copy link
Member Author

Calinou commented Sep 20, 2021

I DM'd Fernando on Twitter and he told me that he really likes the new appearance. This should be good to merge once 3.4 is released 🙂

@aaronfranke aaronfranke self-requested a review October 15, 2021 05:49
@aaronfranke aaronfranke marked this pull request as ready for review November 5, 2021 15:45
@aaronfranke aaronfranke merged commit a29d28f into godotengine:master Nov 5, 2021
@Calinou Calinou deleted the environment-tweak-tonemapping-adjustments branch November 5, 2021 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants