-
-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add run_constrained #84
Conversation
From https://github.com/pydata/xarray/blob/main/ci/requirements/min-all-deps.yml with only netcdf4, h5netcdf, zarr backends specified. Closes pydata/xarray#7467
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2023.02.28.00.25.39
To check my understanding of what this does, this will warn (or error?) when a user tries to install xarray into a environment where one of the listed optional dependencies is out of range from what is listed in |
@jhamman that will error out with a package conflict message. It is, in a way, the best way to control optional dependencies version. |
And when creating a new env, it'll ensure these minimum versions even if the env file doesn't list them? |
Nope. The user must list the optional dependencies. The user can drop the min versions pin though, the solver will do te right thing and respect the run constraints in xarray. |
Right that's what I meant. Sorry for nbeing unclear. The user lists all the packages they want; and the solver will respect xarray's run constraints. Sounds like a feature! We should have a policy on what to list here though: pydata/xarray#7573 Should Xarray limit iris for example? It seems like almost no one is converting xarray datasets to iris or vice versa. |
Yeah. We, conda-forge, should probably add those automatically for python projects in our recipes via grayskull. The problem is that most Python packages use this so loosely that we will probably create packages that are too "rigid" to install rather than more stable solves. TL;DR the xarray metadata is solid and that makes sense here. Doing that for all packages, not so much. |
is pandas 2 safe? |
For the xarray version in this PR? No. But if we are to merge this we would rebase and the latest version of xarray already pins pandas accordingly. |
closed by #93 |
From https://github.com/pydata/xarray/blob/main/ci/requirements/min-all-deps.yml with only netcdf4, h5netcdf, zarr backends specified.
Closes pydata/xarray#7467
Checklist
conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)