xarray-contrib / cf-xarray

an accessor for xarray objects that interprets CF attributes
https://cf-xarray.readthedocs.io/
Apache License 2.0
159 stars 41 forks source link

⚠️ Nightly upstream-dev CI failed ⚠️ #507

Open github-actions[bot] opened 8 months ago

github-actions[bot] commented 8 months ago

Workflow Run URL

Python 3.12 Test Summary ``` cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[True-units-True]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[True-units-False]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[True-None-True]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[True-None-False]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[False-units-True]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[False-units-False]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[False-None-True]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal cf_xarray/tests/test_accessor.py::test_add_canonical_attributes[False-None-False]: AssertionError: assert {'long_name', 'standard_name'} < {'long_name', 'standard_name'} Both sets are equal ```
dcherian commented 5 months ago

@keewis do you know what's going on here?

keewis commented 5 months ago

the failing units test is a bug in pint, which should be fixed and released pretty soon. For the others I'll have to investigate.

dcherian commented 5 months ago

Thanks @keewis . I was only looking for input on pint. The rest just appeared in the latest run!