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

LightmapGI: Lightmap bake creates probes with invalid bright green/magenta colors #77168

Closed
Tracked by #56033
jcostello opened this issue May 17, 2023 · 5 comments
Closed
Tracked by #56033

Comments

@jcostello
Copy link
Contributor

Godot version

v4.1.dev.custom_build [437041a]

System information

Ubuntu with Nvidia 2060

Issue description

Dynamic objects get light from invalid probes

Screenshot from 2023-05-17 15-17-47

Steps to reproduce

Use MRP

Bake the lightmap (in medium quality)
Move the mesh in the editor to get the dynamic lighting.
You will get same result as the picture above

Minimal reproduction project

lightmap.zip

@Calinou
Copy link
Member

Calinou commented May 17, 2023

I don't see a way to resolve this easily. We can't perform a collision check (as that may not always make sense), and generating probes in solid geometry is probably intended behavior (as you'd lose a lot of detail otherwise).

@Calinou Calinou changed the title Lightmap dynamic objects get lighting from invalid probes LightmapGI: Lightmap dynamic objects get lighting from invalid probes May 17, 2023
@jcostello
Copy link
Contributor Author

I don't see a way to resolve this easily. We can't perform a collision check (as that may not always make sense), and generating probes in solid geometry is probably intended behavior (as you'd lose a lot of detail otherwise).

Inside solid geometry probes will be black anyways because they wont get any light. So meshes close to solid geometry will get darker. I don't think that's appropriate. It would be nice at least a way to remove manually probes that are not desirable

@clayjohn
Copy link
Member

clayjohn commented Oct 3, 2023

Since #77169 is fixed by #82068, I guess this issue will be too?

@jcostello
Copy link
Contributor Author

Yes it is. But it will be good in the future to mark probes as invalid if they are inside a geometry (or if they can capture any light)

@Calinou Calinou changed the title LightmapGI: Lightmap dynamic objects get lighting from invalid probes LightmapGI: Lightmap bake creates probes with invalid bright green/magenta colors Nov 10, 2023
@Calinou
Copy link
Member

Calinou commented Nov 10, 2023

This appears to be resolved as of 4.2.beta da0b1eb (Linux, GeForce RTX 4090 with NVIDIA 535.113.01):

image

Fixed MRP (opens without dependency errors, bakes faster with smaller lightmaps – though the issue didn't occur at the original lightmap size either): lightmap_2.zip

Note that dynamic object lighting doesn't update after baking lightmaps until you move or rotate the dynamic object, but this is a separate issue tracked in #77167.

@Calinou Calinou closed this as completed Nov 10, 2023
@Calinou Calinou added this to the 4.2 milestone Nov 10, 2023
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

3 participants