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 ReflectanceCorrector (crefl) for MODIS data #1808
Conversation
I'm not really sure how to test this...ugh I guess I could rewrite all the tests. Darn it. |
Codecov Report
@@ Coverage Diff @@
## main #1808 +/- ##
=======================================
Coverage 92.94% 92.94%
=======================================
Files 265 265
Lines 39180 39191 +11
=======================================
+ Hits 36415 36426 +11
Misses 2765 2765
Flags with carried forward coverage won't be shown. Click here to find out more.
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.
LGTM, although I'm not really familiar with the code I have to say :)
Thanks, just needed a second set of eyes on it. |
* Fix ReflectanceCorrector (crefl) for MODIS data * Update crefl tests to include tbase.hdf based tests * Refactor crefl tests * Remove duplicate imports from crefl tests
I apparently misread my original code when porting the crefl code for reading the DEM data. The file used for MODIS (tbase.hdf) has a different variable name and doesn't have a fill value. This PR fixes that.