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

Implement blend distance to tweak or disable blending between Reflection Probes #4776

Closed
jcostello opened this issue Jun 29, 2022 · 3 comments

Comments

@jcostello
Copy link

Describe the project you are working on

3D environment lighting

Describe the problem or limitation you are having in your project

When working with several Reflection Probes (some of them nested), sometimes you want some details to not blend with the parent reflection probe, so you can have accurate reflection in some areas.

Desire Reflection in the car

image

Undesired Reflection in the car

image

Describe the feature / enhancement and how it helps to overcome the problem or limitation

The idea is to have a blend distance attribute to control when the blending between probes starts or even if they blend (one of the probes should override the other one. Maybe with a priority attribute or with hierarchy

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

Read above

If this enhancement will not be used often, can it be worked around with a few lines of script?

No work around posible

Is there a reason why this should be core and not an add-on in the asset library?

Reflection probes are core

@wareya
Copy link

wareya commented Jun 30, 2022

+1, but commenting to note that I was having similar issues in 3.5 w/r/t controlling how probes blended with the sky (rather than each other), so whatever gets implemented should make sure take that into account as well.

@jcostello
Copy link
Author

+1, but commenting to note that I was having similar issues in 3.5 w/r/t controlling how probes blended with the sky (rather than each other), so whatever gets implemented should make sure take that into account as well.

related to godotengine/godot#61416

@jcostello
Copy link
Author

implemented in godotengine/godot#99958

@AThousandShips AThousandShips added this to the 4.4 milestone Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants