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
Add function to the SLSTR L1 reader to enable correction of VIS radiances. #1401
Conversation
…nce values, as recommended by the latest product notice.
Codecov Report
@@ Coverage Diff @@
## master #1401 +/- ##
==========================================
+ Coverage 90.56% 90.62% +0.06%
==========================================
Files 228 229 +1
Lines 33406 33676 +270
==========================================
+ Hits 30254 30519 +265
- Misses 3152 3157 +5
Continue to review full report at Codecov.
|
DeepCode's analysis on #08b130 found:
Top issues
👉 View analysis in DeepCode’s Dashboard | Configure the bot |
This should be ready for review now. |
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!
The SLSTR instrument is producing data with significant bias in the visible channels.
As defined in the currently-valid product notice, the S5 and S6 channel radiances should be adjusted by between 12 and 20% depending on channel and view. Corrections are also needed for other VIS channels, and these correction factors will be available soon.
In the meantime, this PR enables users to load SLSTR data with the currently-valid radiance correction factors applied, which will produce better data for users requiring the S5 or S6 radiances.
flake8 satpy