Closed djhoese closed 4 days ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.94%. Comparing base (
8bcd427
) to head (aff4f31
). Report is 7 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Totals | |
---|---|
Change from base Build 9520945640: | 0.001% |
Covered Lines: | 51579 |
Relevant Lines: | 53705 |
I need this pull request for my work. Since everyone is going on vacation, I'm going to flex my github permission muscles and push this through. The overall fix is pretty simple, but we should probably have a larger discussion about the "standard practice" being utilized here.
While testing
viirs_edr
with our upcoming release of Polar2Grid 3.1, @kathys noticed that the AWIPS tiled writer fails to handlevalid_range
metadata that is a numpy array. It assumes it is a tuple or something else boolean-able. I can't find any documentation in the Satpy documentation on this, but I'm fairly certain that we tend to recommend either removing metadata fields likevalid_range
or to convert them to tuples. This PR converts theviirs_edr
reader's metadata to a tuple forvalid_range
and now writing to AWIPS tiled files works as expected.