Anutosh Bhat
Anutosh Bhat
> So everyone just saw this on the mailing list: > > Hah! Pretty funny with this parallel conversation going on. Hey @moorepants , that's me and that's my email...
cc @jgranie did you recognize what was the reason behind this error you raised above https://github.com/princeton-vl/infinigen/issues/21#issuecomment-1604434973
What we have (which is the first 3 expansions) is perfectly correct but the last one doesn't work. `Ci(x)` at `-oo` , the expansion should be pretty similar to what...
Yes these questions make sense For eg we have ``` >>> sin(1/x).series(x, oo) 1/(120*x**5) - 1/(6*x**3) + 1/x + O(x**(-6), (x, oo)) ``` Similarly in the expanded form, we should...
Okay I think the nseries methods for these trigonometric integral functions might need some revamping. I think they can be simplified/improved. @arnabnandikgp would you like to look into it ?
Hmm, that makes sense, good ahead and raise a PR for the same.
Let's work in smaller bunch of changes if that makes sense @haru-44 @mohakmalviya To the above comment, I have replied on the issue (https://github.com/sympy/sympy/issues/26207#issuecomment-1951039015). Let's get this in and then...
Merging this but I have updated the PR template and removed the issue number which was mentioned, as there are few things that should still be fixed through subsequent PRs....
Also please fix the code quality. I think you might have to pull the latest changes and rebase on top of it.
Thanks a lot for this pr. We've faced problems with using nsimplify in nseries methods in the past and this PR would help us with that. I shall review this...