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

Fog dark #3178

Closed
lilleyse opened this issue Nov 10, 2015 · 6 comments · Fixed by #3810
Closed

Fog dark #3178

lilleyse opened this issue Nov 10, 2015 · 6 comments · Fixed by #3810

Comments

@lilleyse
Copy link
Contributor

I experienced an issue with the fog when testing the 3d Models Sandcastle. This happens when I first orient the camera into the ground (horizon view), and then start zooming out.

fog-dark

@pjcozzi
Copy link
Contributor

pjcozzi commented Nov 10, 2015

@bagnell can you look at this before the 1.16 release? I suspect this is because of the time of day and the view direction, the blend color is dusk/dawn, but it's strange that it changes when zoomed out.

@pjcozzi
Copy link
Contributor

pjcozzi commented Nov 25, 2015

@bagnell just a reminder that 1.16 is coming up next week. No one has reported running into this yet so we can push if it is really hard to fix.

@bagnell
Copy link
Contributor

bagnell commented Nov 25, 2015

Its either a problem with the atmosphere code or the camera collision with the terrain but I haven't tracked it down. This happens in master too. If you go to the same Sandcastle example in master, keep rotating down and the sky will become super bright.

@lilleyse
Copy link
Contributor Author

In Camera changing

Matrix4.inverse(camera._viewMatrix, camera._invViewMatrix);

to inverseTransformation fixes both fog and shadows. It looks like the camera vectors are a bit wrong, I'll look into this more but at least here is the easy fix.

@lilleyse
Copy link
Contributor Author

Whoops other way around, inverseTransformation to inverse fixes it.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 4, 2016

There was a proposed workaround in #3750.

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