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 handling of powm1 overflow errors #17855

Merged
merged 2 commits into from Jan 26, 2023
Merged

Conversation

mborland
Copy link
Contributor

Reference issue

No open issue. Fixes handling of overflow errors that are on the Boost.Math develop branch found in #17432

What does this implement/fix?

Overflow errors always returned positive infinity. This changes the behavior based on the value of x to return negative infinity such as in the failing test case: x = -INF, y = 3

Additional information

N/A

Co-authored-by: Warren Weckesser <warren.weckesser@gmail.com>
@j-bowhay j-bowhay added defect A clear bug or issue that prevents SciPy from being installed or used as expected scipy.stats C/C++ Items related to the internal C/C++ code base labels Jan 25, 2023
@WarrenWeckesser WarrenWeckesser merged commit e63cc01 into scipy:main Jan 26, 2023
@WarrenWeckesser
Copy link
Member

Thanks @mborland, merged. The existing tests should cover this case.

@mborland mborland deleted the powm1 branch January 26, 2023 04:29
@tylerjereddy tylerjereddy added the backport-candidate This fix should be ported by a maintainer to previous SciPy versions. label Feb 18, 2023
@tylerjereddy
Copy link
Contributor

I think this was a precursor attempt at bug fix prior to gh-17872, so I'll add backport label here as well

@tylerjereddy tylerjereddy modified the milestones: 1.11.0, 1.10.1 Feb 18, 2023
tylerjereddy pushed a commit to tylerjereddy/scipy that referenced this pull request Feb 18, 2023
@tylerjereddy tylerjereddy removed the backport-candidate This fix should be ported by a maintainer to previous SciPy versions. label Feb 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C/C++ Items related to the internal C/C++ code base defect A clear bug or issue that prevents SciPy from being installed or used as expected scipy.stats
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants