-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Ground atmosphere/fog hue, saturation and brightness shifts #7157
Conversation
Thanks for the pull request @bagnell!
Reviewers, don't forget to make sure that:
I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome. 🌍 🌎 🌏 |
@OmarShehata would you like to review? |
@bagnell this looks good to me! I like the updated Sandcastle example. The only failing test is a color test in ModelSpec |
These code changes don't affect anything in model so something else is causing that to fail. |
I'm still seeing a failure on that test. It seems like the epsilon was changed from 5 to 10 here: 13e9f12#diff-dfb89d32e0b83bc7b650d1c64c56bfb5 But even when I check out that commit, |
I should mention that test is also failing in master. |
Right, so it shouldn't impact this PR and perhaps @lilleyse can take a look at it. |
I think we should merge this then since that failing test is unrelated and everything else looks good (I don't have commit access so I can't merge). |
Thanks @OmarShehata for the review. I had fun trying out the updated demo. Once CI passes I'll merge. |
Added
atmosphereHueShift
,atmosphereSaturationShift
, andatmosphereBrightnessShift
properties toGlobe
which shift the color of the ground atmosphere to match the hue, saturation, and brightness shifts of the sky atmosphere.Fixes #4195.