search

yt-project / yt

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

switch xradar pin to arm-pyart pin in tests #5048

Closed chrishavlin closed 1 week ago

chrishavlin commented 1 week ago

new arm-pyart release is out. So close #5044 ... and hopefully #5047 ??

chrishavlin commented 1 week ago

I just realized that anything that is added to the pyproject.toml test target will always be installed, even in the minimal test suite (the ci_install.sh script will do a python -m pip install -e ".[test]" -r minimal_requirements.txt for the minimal test). So having arm-pyart (or previously xradar) in the test target will mean that it will always be installed (which will include xarray and lots of other dependencies). That doesn't seem ideal... I'm tempted to just pin arm-pyart in the frontend requirements instead?

chrishavlin commented 1 week ago

ok, just pushed a new version to just bump the min arm-pyart version. It's a pretty big jump in versions from what we had... but realistically I think it'd be tough to get a working environment at v<1.19.2 with all the shuffling of the arm-pyart dependencies that happened.