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

Vulkan: ReflectionProbe rendering considers extents as (1, 1, 1) when set to the default value of (10, 10, 10) #66624

Closed
Tracked by #66628
Calinou opened this issue Sep 29, 2022 · 0 comments · Fixed by #71910

Comments

@Calinou
Copy link
Member

Calinou commented Sep 29, 2022

Godot version

4.0.beta1 (can also be reproduced in all 4.0 alphas)

System information

Fedora 36, GeForce GTX 1080 (NVIDIA 515.65.01)

Issue description

ReflectionProbe rendering considers extents as (1, 1, 1) when set to the default value of (10, 10, 10). This is a regression from #49953.

#55178 should resolve this, but I'm opening an issue nonetheless to keep track.

Default extents ((10, 10, 10))

Floor doesn't appear in the reflection as it's located outside the actual rendered extents ((1, 1,1), the old default value).

2022-09-29_23 49 14

Custom extents ((10.1, 10.1, 10.1))

2022-09-29_23 49 29

Steps to reproduce

  • Add a MeshInstance node with a BoxMesh and increase its size to act as a floor.
  • Add a MeshInstance node with a SphereMesh and a reflective material (roughness = 0, metallic = 1). Move it above the BoxMesh.
  • Add ReflectionProbe node in the middle of the SphereMesh.
  • Save the scene then use Scene > Reload Saved Scene. Notice how the floor doesn't appear in the sphere's reflection.
  • Change the ReflectionProbe's extents to any value that is different from (10, 10, 10). Notice how the floor now appears in the sphere's reflection.

Minimal reproduction project

test_reflection_probe_default_extents.zip

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.

3 participants