-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
add test to close 14037 #19805
add test to close 14037 #19805
Conversation
✅ Hi, I am the SymPy bot (v160). I'm here to help you write a release notes entry. Please read the guide on how to write release notes.
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.
|
Codecov Report
@@ Coverage Diff @@
## master #19805 +/- ##
=============================================
+ Coverage 75.693% 75.707% +0.013%
=============================================
Files 658 664 +6
Lines 171307 172374 +1067
Branches 40441 40653 +212
=============================================
+ Hits 129669 130500 +831
- Misses 35973 36162 +189
- Partials 5665 5712 +47 |
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.
Just add this testcase as well to series/test_series.py
:
In [2]: (sin(x**50)/x**51).series(x, n=0)
Out[2]:
1
- + O(1)
x
@sachin-4099 I added the test, however it is causing an assertion error and I can't seem to figure out why. I ran the command on the master and it worked fine, so I was wondering if it could possibly be a syntax issue or something else. The contents of the test are |
This will work: |
Looks good to me. This can be merged @jksuom. |
Thanks. |
References to other Issues or PRs
Fixes #14037
Release Notes
NO ENTRY