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
Handle Mul.is_imaginary for infinite values #18192
Conversation
✅ Hi, I am the SymPy bot (v149). 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.6. Note: This comment will be updated with the latest check if you edit the pull request. You need to reload the page to see it. Click here to see the pull request description that was parsed.
Update The release notes on the wiki have been updated. |
@oscarbenjamin could you please look into this? |
from travis' log:
|
If you find where that failed test is you can get the code from there to see what has changed. Presumably the bessel function now evaluates differently for |
Okay, I'll look into it. Happy new year btw! |
The release notes should be something like There is no need to mention the changes to tests, they do not affect the users. |
@Sc0rpi0n101 Thanks for the advice.
I'll consider that for the future PRs. |
This should fix the failing tests.
|
Thank you, @V1krant. :) |
@oscarbenjamin @czgdp1807 This diff indeed fixes the failing test.
There's no such failure on running the tests locally. |
That's a bug in pip |
How can I fix that? |
Codecov Report
@@ Coverage Diff @@
## master #18192 +/- ##
============================================
+ Coverage 75.311% 75.32% +0.009%
============================================
Files 635 637 +2
Lines 167071 167069 -2
Branches 39422 39416 -6
============================================
+ Hits 125823 125838 +15
+ Misses 35711 35689 -22
- Partials 5537 5542 +5 |
Looks good. Thanks all. |
Thanks all. |
References to other Issues or PRs
Fixes #17556
Brief description of what is fixed or changed
Other comments
Release Notes
Mul.is_imaginary
to support infinite values.