Skip to content

Revert pytest: use importlib mode by default #28690

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

Merged
merged 1 commit into from
May 13, 2025

Conversation

jakevdp
Copy link
Collaborator

@jakevdp jakevdp commented May 12, 2025

Rolls back #28650, because it broke our nightly tests.

Fixes #28671.

@jakevdp jakevdp self-assigned this May 12, 2025
@jakevdp jakevdp added the pull ready Ready for copybara import and testing label May 12, 2025
@jakevdp jakevdp requested a review from dfm May 12, 2025 15:35
Copy link
Contributor

@dfm dfm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't read all the linked details on #28650, but I wonder if it's worth trying to re-land that change after trying to diagnose the nightly failures? WDYT?

(PS I'm happy to do that debugging if you think it's worthwhile!)

@olupton
Copy link
Contributor

olupton commented May 12, 2025

Just flagging #28650 (comment) here too.

@jakevdp
Copy link
Collaborator Author

jakevdp commented May 13, 2025

I'm going to submit this.

Dan - if you'd like to explore rolling this forward, please do!

@copybara-service copybara-service bot merged commit f62b790 into jax-ml:main May 13, 2025
23 checks passed
@jakevdp jakevdp deleted the revert-28650 branch May 13, 2025 16:58
@olupton
Copy link
Contributor

olupton commented May 14, 2025

I'm going to submit this.

Dan - if you'd like to explore rolling this forward, please do!

@dfm I posted some other info #28650 (comment) -- I'll leave it to you to try and roll this forwards if that's OK?

@dfm
Copy link
Contributor

dfm commented May 14, 2025

@olupton — I'm happy to! It might be useful if you would open a new issue with the relevant details summarized? Would you be able to do that?

@olupton
Copy link
Contributor

olupton commented May 14, 2025

@olupton — I'm happy to! It might be useful if you would open a new issue with the relevant details summarized? Would you be able to do that?

Done in #28740

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kokoro:force-run pull ready Ready for copybara import and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed build: CI - with Numpy/Scipy nightly wheels (nightly)
3 participants