Skip to content
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

BUG: Fix bounds checking for random.logseries #22594

Merged
merged 1 commit into from Nov 15, 2022

Conversation

charris
Copy link
Member

@charris charris commented Nov 15, 2022

Backport of #22450.

Logseries previously did not enforce bounds to be strictly exclusive for the upper bound, where it leads to incorrect behavior.

The NOT_NAN check is removed, since it was never used: The current bounded version always excludes NaNs.

Logseries previously did not enforce bounds to be strictly exclusive
for the upper bound, where it leads to incorrect behavior.

The NOT_NAN check is removed, since it was never used: The current bounded
version always excludes NaNs.
@charris charris added this to the 1.23.5 release milestone Nov 15, 2022
@charris charris merged commit 3ca02ce into numpy:maintenance/1.23.x Nov 15, 2022
40 checks passed
@charris charris deleted the backport-22450 branch November 15, 2022 19:31
@charris charris changed the title BUG: Fix boundschecking for random.logseries BUG: Fix bounds checking for random.logseries Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants