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

Set Environment's Ao Channel Effect to 1.0 by default #43960

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Nov 29, 2020

"Ao Channel Affect" was defaulting to 0, giving unexpected results. This meant that as soon as you set an AO texture on a material, there will no longer be SSAO on that material.

This led many users to think that Godot didn't support using SSAO and AO textures at the same time.

This closes #43957.

"Ao Channel Affect" was defaulting to 0, giving unexpected results.
This meant that as soon as you set an AO texture on a material,
there will no longer be SSAO on that material.

This led many users to think that Godot didn't support using
SSAO and AO textures at the same time.

This closes godotengine#43957.
@Calinou Calinou requested a review from a team as a code owner November 29, 2020 16:38
@Calinou Calinou added this to the 4.0 milestone Nov 29, 2020
@fire fire self-requested a review December 1, 2020 18:52
@fire
Copy link
Member

fire commented Dec 1, 2020

Should this pr also handle the importer changes?

@fire fire changed the title Set Environment's Ao Channal Effect to 1.0 by default Set Environment's Ao Channel Effect to 1.0 by default Dec 2, 2020
@Calinou
Copy link
Member Author

Calinou commented Jan 8, 2021

Superseded by #45023.

-SSAO ao affect is gone because this confused users, min(ao,ssao) now used, should work in most cases.

This pull request could perhaps be remade for the 3.2 branch.

@Calinou Calinou closed this Jan 8, 2021
@Calinou Calinou removed this from the 4.0 milestone Jan 8, 2021
@Calinou Calinou deleted the tweak-default-environment-ao-channel-affect branch March 31, 2021 09:44
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.

"Ao Channel Affect" defaults to 0, giving unexpected results (fixed in master via different AO calculation)
2 participants