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
MAINT: Move to new numpy.random API in test_*.py #16086
base: main
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
👋 Thank you for your draft pull request! Do you know that you can use |
Out of scope here but I just want to link for future self... #11566 |
@@ -553,8 +553,8 @@ def test_slice_to_quantity(self): | |||
""" | |||
Regression test for https://github.com/astropy/astropy/issues/2003 | |||
""" | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MridulS, what if you defined a rng
fixture in the main conftest.py
, then you can just add an rng
argument to any pytest function to get the rng. Should save a lot of lines!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for units
. However, see my other comment for a way to simplify all the tests using a rng
pytest fixture.
This PR is another go at #12669 and this builds on top of #13747. With this PR we can start addressing #11687
This PR does not make any API changes! This just edits the tests file to update them to the old numpy random generator API.
There is a ruff numpy rule, so I am trying to make it happy here. For certain tests I had to play around with seeds, atol/rtol. I still need to work a bit more on this.
Once this is in we can start looking at adding arguments to expose numpy RNG instance for functions that depend on internal random state. Once this is in we can also start deprecating
NumpyRNGContext
.