Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork1.2k
Make sure setuptools is compatible with PEP 639#10512
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
DocOtak commentedJul 7, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I think this error is saying that the thing in
or
which is a link to the file itself |
DimitriPapadopoulos commentedJul 8, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Right, requiring However, if I understand you correctly, this is not compatible with Numpy 2.2: xarray/asv_bench/asv.conf.json Line 63 in22bd964
We'll probably have to partially revert#10300 and fall back to pre-PEP639 format. Wait, no, according to#10355 there's more to that. I guess the real question is why I saw this issue surface recently in CI tests? |
PEP 639 requires setuptools ≥ 77.It seems customary to require setuptools ≥ 77.0.2 which irons out somebugs related to PEP 639 support.
281ea27
to7defc56
Compare
The error you mention above is from the asv benchmarks (only running CI after adding the corresponding label), for which Xarray is installed with |
Uh oh!
There was an error while loading.Please reload this page.
Instead rely onproject.license
inpyproject.toml
.Let's see if this fixes this CI error:
whats-new.rst
api.rst