search

yt-project / yt

Main yt repository
http://yt-project.org
Other
469 stars 280 forks source link

TST: failing to load `pyart` #5041

Closed neutrinoceros closed 2 weeks ago

neutrinoceros commented 2 weeks ago

The test pipeline on Jenkins appears to be broken as an ImportError is raised when trying to lazy-load pyart

https://tests.yt-project.org/job/yt_py310_git/8571/

I have reported the problem upstream https://github.com/ARM-DOE/pyart/issues/1680 In the meantime we can add datatree as an optional requirement on our side as a temporary measure.

kmuehlbauer commented 2 weeks ago

@neutrinoceros Heads-up, datatree was merged into xarray codebase recently.

To use xarray-datatree (as in the test) you would also need to pin xarray to <=2024.9.0 and xradar <=to 0.7.0.

xradar 0.7.0 has a pin for xarray <= 2024.9.0, uses old datatree from xarray-contrib/datatree xradar 0.8.0 has a pin for xarray >= 2024.10.0, uses new datatree from xarray codebase

HTH

neutrinoceros commented 2 weeks ago

marking this as a blocker since we cannot merge anything until there's at least a workaround for this issue

chrishavlin commented 2 weeks ago

I can take a look at modifying the deps on the yt side to get tests to pass.

neutrinoceros commented 2 weeks ago

fixed in #5042