-
Notifications
You must be signed in to change notification settings - Fork 297
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
Adjust default blending in DayNightCompositor #1064
Adjust default blending in DayNightCompositor #1064
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1064 +/- ##
==========================================
- Coverage 88.09% 88.09% -0.01%
==========================================
Files 191 191
Lines 29439 29438 -1
==========================================
- Hits 25935 25934 -1
Misses 3504 3504 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My gut reaction was "why change it, it looks fine already" until I actually saw the comparison. Yeah, this looks better.
Have you tried it with other composite combinations? Like with a true_color on the day side? I'm wondering if the edges are worse...wait this change just gives more of the night image, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for fixing this.
Good image. That's what I was hoping for; an image with a very dark image on one side and a bright image on the other. Looks good. |
How does it look like if you take a night-only composite on the night side, eg night_microphysics, does it still look ok ? |
Looks really good @pnuu ! |
This PR adjusts the default Sun zenith angle (SZA) interval where the overlapping day and night products are blended from
85.0 ... 95.0
to85.0 ... 88.0
.The default limit of SZA correction in Satpy is the same 88.0 degrees. If the upper limit is higher, there will be a dark band between the products.
The original version:
Proposed version:
flake8 satpy