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
Fix 'virr_l1b' reader when slope attribute is 0 #800
Conversation
Update project
…l-master # Conflicts: # satpy/readers/virr_l1b.py
Codecov Report
@@ Coverage Diff @@
## master #800 +/- ##
==========================================
+ Coverage 82.26% 82.27% +<.01%
==========================================
Files 159 159
Lines 22998 23003 +5
==========================================
+ Hits 18920 18925 +5
Misses 4078 4078
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.
I'm guessing by reading this that Slope
in the file can either be an array or a single value? Or is the single value (scalar) only coming from your tests?
What happened to what we talked about in person with slope = slope.where(slope != 0, 1)
? Would it be cleaner to make a small helper function for this check so it doesn't have to be repeated so many times? Or maybe that's overkill.
If slope in the file being read is 0, convert slope to 1 (This came about from Longitude in FY3B files).
flake8 satpy
AUTHORS.md
if not there already