-
Notifications
You must be signed in to change notification settings - Fork 974
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
Use numpy._core
instead of numpy.core
#3247
Conversation
The issue here is that |
good point! The very first version that includes What about something like np_core = np._core if version.parse(np.__version__) >= version.parse("2.0.0") else np.core alternatively we can do something like np_core = getattr(np, "_core", np.core) but I find it less intuitive to know when the condition is met or not, and we'll probably forget to remove this if/else once we've increased the min version of numpy EDIT: min version is actually 2.0.0 |
wait it seems like EDIT: version threshold is actually 2.0.0 (not 1.26.1). Editing my above comment |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
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.
Thanks for fixing the warning.
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.
Nice thanks !
What does this PR do?
We've been getting this warning:
Before submitting
Pull Request section?
to it if that's the case.
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
@muellerzr @BenjaminBossan @SunMarc