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
zero raised to power -infinity gives zoo #20212
Conversation
✅ Hi, I am the SymPy bot (v161). I'm here to help you write a release notes entry. Please read the guide on how to write release notes. Your release notes are in good order. Here is what the release notes will look like:
This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.7. Click here to see the pull request description that was parsed.
Update The release notes on the wiki have been updated. |
Codecov Report
@@ Coverage Diff @@
## master #20212 +/- ##
==============================================
+ Coverage 64.385% 75.706% +11.321%
==============================================
Files 671 671
Lines 174144 174146 +2
Branches 41093 41094 +1
==============================================
+ Hits 112123 131840 +19717
+ Misses 55599 36551 -19048
+ Partials 6422 5755 -667 |
Review pls |
sympy/core/tests/test_power.py
Outdated
|
||
def test_issue_19572(): | ||
assert 0 ** -oo is zoo | ||
assert power(0, -oo) is zoo |
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.
Can you add this to test_zero
above rather than a new test function? You can include the url for the issue in a comment.
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.
Yes sure
@oscarbenjamin Transferred the test cases to test_zero() |
Is it good to go ? |
Yes, looks good |
References to other Issues or PRs
Fixes #19572
Brief description of what is fixed or changed
Earlier 0 ** -oo gave 0 instead of zoo
Other comments
Release Notes