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

[Bug] Metallic materials aren't affected by Skylight #2687

Open
Myterian opened this issue Jun 4, 2024 · 5 comments
Open

[Bug] Metallic materials aren't affected by Skylight #2687

Myterian opened this issue Jun 4, 2024 · 5 comments

Comments

@Myterian
Copy link

Myterian commented Jun 4, 2024

Issue description:
Skylight don't affect metallic materials, making them unnaturally dark in shadowed areas. Watch the cannon in the front.

No Skylight
grafik

A bit Skylight
grafik

A lot Skylight
grafik

Steps to reproduce:

  1. Add an object to your scene
  2. Create a material, set Metalness to 1
  3. Create Skylight in scene
  4. Play with Skylight brightness and observe

Minimal reproduction project:
Basic

Flax version:
1.8.6512.2

@mafiesto4 mafiesto4 added this to the 1.9 milestone Jun 5, 2024
@mafiesto4
Copy link
Member

Is there an environment probe in that scene? Metallic materials use reflection probes to get lighting (diffuse is mostly black).

@Myterian
Copy link
Author

Myterian commented Jun 5, 2024

grafik

So, reflections probes solved the problem, you can see the shadowed parts now. There is a new issue, tho, as that the reflection probe makes everything shiny or 'wet', even on completly rough surfaces and especially in shadowed areas. Like the cannons appear shiny, but have the same roughness values as the wood crate next to them.

Do you think this could be fixed with by making metalic materials use things like skylight instead of reflections or is this as good as it gets?

@mafiesto4
Copy link
Member

Maybe we could add an optional intensity slider to skylight for mixing with reflections (can scale the alpha channel of the output light buffer which is used for reflections intensity mixing). Or shadowed skylight would improve the quality in shadowed areas: #1189

@mafiesto4 mafiesto4 modified the milestones: 1.9, Future Jun 8, 2024
@Myterian
Copy link
Author

Myterian commented Jun 8, 2024

Sounds good

@Myterian
Copy link
Author

I just found another good reason for skylights in metallic materials:
Screenshot 2024-06-11 200529

In dynamic day and night scenes, metallic parts are unaffected by the light changes. Which makes them stick out

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

2 participants