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

OSM buildings are very shiny #12363

Open
jjspace opened this issue Dec 5, 2024 · 3 comments
Open

OSM buildings are very shiny #12363

jjspace opened this issue Dec 5, 2024 · 3 comments

Comments

@jjspace
Copy link
Contributor

jjspace commented Dec 5, 2024

simplescreenrecorder-2024-12-05_11.10.32.mp4

According to a discussion on Slack the OSM buildings have always been shiny it just wasn't very obvious in the past. Something in the 1.124 release however has made if very obvious and created the sort of "star" pattern you can see in the video above. The video also shows switching back to 1.123 to show that the pattern was there before just much much more subtle so it didn't look like a portal to another dimension.

I'm assuming this is a result of some of the lighting/environment map changes that we've added to help general visual quality. I haven't tried to do a git bisect or anything yet to get to the root commit that caused this change.

I'm hoping this maay be an easy change in how we create the tileset in the OSM buildings helper but even if we can we should probably look into fixing it. This will be very obvious for anyone using the OSM buildings and moving the camera around (like in our beginner flight tracker tutorial)

@dkbraig
Copy link
Contributor

dkbraig commented Dec 10, 2024

While testing OSM building updates, I noticed some visual weirdness in the OSM building materials I am not sure if this is related or a separate issue

OSM Materials close up

@ggetz
Copy link
Contributor

ggetz commented Dec 10, 2024

@dkbraig It also looks like the cubemap for image-base lighting is rotated on it side for that dataset. The green color should be coming from beneath the building, not to the side like in this image:

image

@ggetz
Copy link
Contributor

ggetz commented Dec 10, 2024

For the above, #12310 may be the cause.

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