Skip to content
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

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

Closed
github-actions bot opened this issue Jul 11, 2023 · 7 comments
Closed

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

github-actions bot opened this issue Jul 11, 2023 · 7 comments
Labels
CI Continuous Integration tools

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2023

Workflow Run URL

Python 3.10 Test Summary

@github-actions github-actions bot added the CI Continuous Integration tools label Jul 11, 2023
@keewis
Copy link
Collaborator

keewis commented Jul 14, 2023

the interpolate tests fail because of pandas-dev/pandas#53962

@keewis
Copy link
Collaborator

keewis commented Jul 18, 2023

we now get a lot of TypeErrors related to the addition of a datetime.timedelta to a cftime.datetime object. Interestingly enough, cftime itself doesn't appear to have been changed recently, so we might need more debugging to figure out which other dependency changed. Edit: this appears to have been pandas

cc @spencerkclark

@keewis
Copy link
Collaborator

keewis commented Jul 28, 2023

the numpy representation issues seem related to NEP 51 (the actual change was merged in numpy/numpy#22449)

@spencerkclark
Copy link
Member

we now get a lot of TypeErrors related to the addition of a datetime.timedelta to a cftime.datetime object. Interestingly enough, cftime itself doesn't appear to have been changed recently, so we might need more debugging to figure out which other dependency changed. Edit: this appears to have been pandas

Thanks for the ping. I'll try and look into this soon.

@spencerkclark
Copy link
Member

After some investigation, the cftime-related failures appear to be an upstream issue related to the interaction of the official release of Cython 3.0.0 a couple weeks ago, and arithmetic operations in cftime. See discussion in Unidata/cftime#271; I think we did not see this before, because Cython 3.0.0 was in beta.

@spencerkclark
Copy link
Member

The cftime-related failures should hopefully be fixed in the next run now that Unidata/cftime#305 has been merged.

@keewis
Copy link
Collaborator

keewis commented Aug 10, 2023

The ComplexWarning import errors are from scipy but caused by a change in numpy. scipy has already adapted to that but will only upload a new build this weekend, so this should go away soon.

There's also a few AttributeErrors for iris.cube, but I don't really understand those... we have the same version of iris as for previous runs that didn't fail, so maybe this is related to the numpy / scipy issue?

Edit: both of them went away, so either the iris was intermittent, or it was related to the numpy rename

@keewis keewis mentioned this issue Aug 10, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration tools
Projects
None yet
Development

No branches or pull requests

3 participants