pytroll / satpy

Python package for earth-observing satellite data processing
http://satpy.readthedocs.org/en/latest/
GNU General Public License v3.0
1.06k stars 292 forks source link

Fix the viirs EDR tests for newer xarray #2774

Closed mraspaud closed 6 months ago

mraspaud commented 6 months ago

This PR fixes the Viirs EDR tests for new xarray.

Dtype checks were failing because of the test file generation was being given float64 scale and offset parameters to the encoding.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.92%. Comparing base (3aa2f79) to head (24109a1). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2774 +/- ## ======================================= Coverage 95.92% 95.92% ======================================= Files 377 377 Lines 53535 53535 ======================================= Hits 51356 51356 Misses 2179 2179 ``` | [Flag](https://app.codecov.io/gh/pytroll/satpy/pull/2774/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | Coverage Δ | | |---|---|---| | [behaviourtests](https://app.codecov.io/gh/pytroll/satpy/pull/2774/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | `4.10% <0.00%> (ø)` | | | [unittests](https://app.codecov.io/gh/pytroll/satpy/pull/2774/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | `96.02% <100.00%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8524637244

Details


Totals Coverage Status
Change from base Build 8450068387: 0.0%
Covered Lines: 51228
Relevant Lines: 53358

💛 - Coveralls