Closed yukaribbba closed 3 months ago
pre-commit.ci autofix
@djhoese One of the CI test keeps failing on macos-latest, 3.9, false
. I don't know what's going on.
=========================== short test summary info ============================
FAILED satpy/tests/reader_tests/test_sar_c_safe.py::TestSAFEGRD::test_read_lon_lats - AssertionError:
Not equal to tolerance rtol=1e-07, atol=0
Mismatched elements: 3 / 256 (1.17%)
Max absolute difference: 4.98866104e-09
Max relative difference: 2.33333333
x: array([[ 2.081090e-16, 5.91[666](https://github.com/pytroll/satpy/actions/runs/8872941895/job/24358006466?pr=2795#step:9:667)7e-01, 9.097222e-01, 1.000000e+00,
9.083333e-01, 6.805556e-01, 3.625000e-01, -1.110223e-16,
-3.611111e-01, -6.750000e-01, -8.958333e-01, -9.777778e-01,...
y: array([[ 3.794929e-16, 5.916[667](https://github.com/pytroll/satpy/actions/runs/8872941895/job/24358006466?pr=2795#step:9:668)e-01, 9.097222e-01, 1.000000e+00,
9.083333e-01, 6.805556e-01, 3.625000e-01, 8.326673e-17,
-3.611111e-01, -6.750000e-01, -8.958333e-01, -9.777778e-01,...
= 1 failed, 2541 passed, 4 skipped, 5 xfailed, 1 xpassed, 443 warnings in 352.85s (0:05:52) =
Yes. This is a known problem with a scipy update causing a test failure. It is being looked at.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.94%. Comparing base (
3b9c04e
) to head (f48d740
). Report is 53 commits behind head on main.:exclamation: Current head f48d740 differs from pull request most recent head d04bd36
Please upload reports for the commit d04bd36 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Well it's kinda annoying here that there are no genuine differences inside the datasets but they are still organized in different forms. So I think merging them into one could be a little bit messy for the users...
Yeah MERSI and AGRI were the two instruments I think of with designs like that (new version of the instrument, but completely different numbering/ordering of channels).
pre-commit.ci autofix
pre-commit.ci autofix
@djhoese Some of the CI failed again. But at least the tests passed.
I restarted the CI. Very odd error from the test coverage reporting tool so let's see if it fixes itself...
Failed again....
Looks like a major release of the coveralls python package causing issues: https://pypi.org/project/coveralls/
I have appointments half of today, but I'll try debugging later.
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.
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
satpy/readers/glm_l2.py | 1 | 98.55% | ||
satpy/readers/ascat_l2_soilmoisture_bufr.py | 1 | 97.44% | ||
satpy/tests/multiscene_tests/test_save_animation.py | 1 | 99.1% | ||
satpy/tests/reader_tests/_li_test_utils.py | 1 | 99.26% | ||
satpy/dataset/metadata.py | 1 | 99.26% | ||
satpy/tests/reader_tests/test_li_l2_nc.py | 1 | 99.76% | ||
satpy/tests/reader_tests/test_ascat_l2_soilmoisture_bufr.py | 1 | 97.59% | ||
satpy/readers/mws_l1b.py | 1 | 98.52% | ||
satpy/tests/reader_tests/test_avhrr_l1b_gaclac.py | 1 | 99.66% | ||
satpy/tests/reader_tests/test_ahi_hsd.py | 1 | 99.67% | ||
<!-- | Total: | 570 | --> |
Totals | |
---|---|
Change from base Build 9157044280: | -0.009% |
Covered Lines: | 51524 |
Relevant Lines: | 53658 |
Datasets for test: https://drive.google.com/file/d/1cCoC14AioC2l34jFfxa19F88k5NXuqpi/view?usp=sharing
FY-3A:
_FY3B:
_FY3C: