-
Notifications
You must be signed in to change notification settings - Fork 103
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
Adds support for numpy>=2.0.0
#1829
Conversation
User metrics figure
…t is not in a git repo
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #1829 +/- ##
===========================================
- Coverage 38.53% 38.48% -0.05%
===========================================
Files 206 207 +1
Lines 47992 48151 +159
===========================================
+ Hits 18494 18532 +38
- Misses 29498 29619 +121 ☔ View full report in Codecov by Sentry. |
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.
thx for this!
xsize = slit_righ - slit_left | ||
#nsamp = np.ceil(np.median(xsize)) # JFH Changed 07-07-19 | ||
nsamp = np.ceil(xsize.max()) | ||
if nobj == 0: |
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.
doesn't look like numpy issues, so I assume some other
bits and pieces slipped in. which is fine
Tests pass, except for the one expected failure.
Merging! |
Accommodates changes in numpy version 2.0.
According to this, these changes should not affect use by numpy <2.0.0, so I haven't included an increment in the dependency version requirements yet. But we should likely warn users to be aware of this change and encourage them to upgrade numpy once this is released.
I'll re-run the dev-suite on this and post the results, but we should expect some
bok_bc
failures that we need to fix before tagging a new release.