-
-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
ENH: Implement correct scalar and integer overflow errors for NEP 50 #21875
Commits on Oct 12, 2022
-
ENH: Implement safe integers and weak python scalars for scalars
This requires adding a path that uses the "normal" Python object to dtype conversion (setitem) function, rather than always converting to the default dtype.
Configuration menu - View commit details
-
Copy full SHA for 409cccf - Browse repository at this point
Copy the full SHA 409cccfView commit details -
Configuration menu - View commit details
-
Copy full SHA for d894650 - Browse repository at this point
Copy the full SHA d894650View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0a0f96f - Browse repository at this point
Copy the full SHA 0a0f96fView commit details -
MAINT: Use Python integers for int to array of uint32 calculation
The new weak promotion preserves the original type, this makes the `//= 2**32` fail for certain inputs. The alternative would be typing that as `np.int64(2**32)`, but using Python integers seems easier and cleaner. The code was effectively OK before, since the inputs were guaranteed signed (or Python integers) at that point and 2**32 would have been considered like a NumPy `int64`. (Which would be an alternative fix.)
Configuration menu - View commit details
-
Copy full SHA for 89cfcf0 - Browse repository at this point
Copy the full SHA 89cfcf0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2143cca - Browse repository at this point
Copy the full SHA 2143ccaView commit details -
BUG: Ensure new-style promotion is not accidentally used for some ints
This is the *actual* correct fix for the test adaptations. The test adaptations should only be necessary when running in weak-promotion mode, but they are NOT doing that currently.
Configuration menu - View commit details
-
Copy full SHA for fed11cd - Browse repository at this point
Copy the full SHA fed11cdView commit details -
Configuration menu - View commit details
-
Copy full SHA for d47b09c - Browse repository at this point
Copy the full SHA d47b09cView commit details -
Configuration menu - View commit details
-
Copy full SHA for c4f1419 - Browse repository at this point
Copy the full SHA c4f1419View commit details -
TST: Avoid possible warning from unnecessary cast with uninitialized …
…values This should be fixed in `choose` to not do the unnecessary cast, see numpygh-22237.
Configuration menu - View commit details
-
Copy full SHA for d53a2b6 - Browse repository at this point
Copy the full SHA d53a2b6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0375d16 - Browse repository at this point
Copy the full SHA 0375d16View commit details -
TST: Cover additional NEP 50 scalar paths
Especially adding coverage also for the power operator which does not share its code perfectly.
Configuration menu - View commit details
-
Copy full SHA for 4ecc035 - Browse repository at this point
Copy the full SHA 4ecc035View commit details -
Configuration menu - View commit details
-
Copy full SHA for d9da02d - Browse repository at this point
Copy the full SHA d9da02dView commit details -
Configuration menu - View commit details
-
Copy full SHA for ee3c20b - Browse repository at this point
Copy the full SHA ee3c20bView commit details