Opened 11 years ago
Closed 11 years ago
#8237 closed defect (fixed)
Sage does not recognize Maxima's complex infinity
Reported by: | robert.marik | Owned by: | burcin |
---|---|---|---|
Priority: | major | Milestone: | sage-4.4.2 |
Component: | symbolics | Keywords: | |
Cc: | robert.marik | Merged in: | sage-4.4.2.alpha0 |
Authors: | Burcin Erocal | Reviewers: | Karl-Dieter Crisman, Ross Kyprianou, Robert Mařík |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
As subject says:
sage: maxima('inf').sage() +Infinity sage: maxima('infinity').sage() +Infinity
From Maxima manual
Constant: inf inf represents real positive infinity. Constant: infinity infinity represents complex infinity. Constant: minf minf represents real minus (i.e., negative) infinity.
As a cosequence, Sage fails to evaluate limit of 1/x at x=0. Maxima gives correct result (complex infinity)
sage: maxima('limit(1/x,x,0)') infinity sage: maxima('limit(1/x,x,0)').sage() +Infinity sage: limit(1/x,x=0) +Infinity sage: maxima('limit(1/x,x,0,plus)') inf sage: maxima('limit(1/x,x,0,plus)').sage() +Infinity
Attachments (3)
Change History (18)
comment:1 Changed 11 years ago by
comment:2 Changed 11 years ago by
- Owner changed from burcin to robert.marik
- Summary changed from Sage does not recognize Maxima's complex ininity to Sage does not recognize Maxima's complex infinity
comment:3 Changed 11 years ago by
- Owner changed from robert.marik to burcin
comment:4 Changed 11 years ago by
Right now, there doesn't seem to be a lot of distinction in Sage between unsigned and signed infinity, though of course as you point out there should be. From sage.rings.infinity.py:
Note: the shorthand oo is predefined in Sage to be the same as +Infinity in the infinity ring. It is considered equal to, but not the same as Infinity in the UnsignedInfinityRing:: sage: oo +Infinity sage: oo is InfinityRing.0 True sage: oo is UnsignedInfinityRing.0 False sage: oo == UnsignedInfinityRing.0 True
There is unsigned_infinity, but the following seems problematic:
sage: unsigned_infinity Infinity sage: Infinity +Infinity
What the heck?
comment:5 Changed 11 years ago by
- Milestone set to sage-4.3.3
- Status changed from new to needs_review
I uploaded a patch, please review.
comment:6 Changed 11 years ago by
- Reviewers set to Karl-Dieter Crisman
- Status changed from needs_review to needs_work
Looks good, but does it duplicate some of lines 1841ff of sage/calculus/calculus.py?
from sage.rings.infinity import infinity, minus_infinity register_symbol(infinity, dict(maxima='inf')) register_symbol(minus_infinity, dict(maxima='minf'))
Since
sage: type(infinity) <class 'sage.rings.infinity.PlusInfinity'> sage: type(SR(infinity)) <type 'sage.symbolic.expression.Expression'>
my guess is that, at least for completeness, calculus.py should also import unsigned_infinity and have a line added with
register_symbol(unsigned_infinity, dict(maxima='infinity'))
Also, my taste in doctests is to also include the original example, not (only) the underlying cause:
sage: limit(1/x,x=0) Infinity sage: limit(1/x,x=0,dir='above') +Infinity sage: limit(1/x,x=0,dir='below') -Infinity
which of course works great now. These are very minor quibbles, of course, but might as well be done.
Also, in doctesting it doesn't like sage: sage: as the prefix (though one could argue this is a bug itself), and
devel/sage/sage/calculus/functional.py", line 313: sage: lim(1/x, x=0) Expected: +Infinity Got: Infinity
comment:7 Changed 11 years ago by
- Cc robert.marik added
comment:8 Changed 11 years ago by
- Status changed from needs_work to needs_review
I was trying to get the first patch out quickly, so it ended up being too sloppy. I hope attachment:trac_8237-maxima_infinity.take2.patch is cleaner. :)
I think it's more natural to put the maxima conversions for different infinities in sage/symbolic/constants.py
where all the other constants are declared, so I removed the lines in calculus.py
. I also added doctests with limit(1/x, ...)
.
comment:9 Changed 11 years ago by
- Reviewers changed from Karl-Dieter Crisman to Karl-Dieter Crisman, Ross Kyprianou
By exercising a number of arithmetic use cases, viz.
for k in (1, 1.0, 1/2, x, 1+I, -1, -1.0, -1/2, -x, -1-I): print k, k + Infinity , k + +Infinity, k + -Infinity for k in (1, 1.0, 1/2, x, 1+I, -1, -1.0, -1/2, -x, -1-I): print k, Infinity -k , +Infinity -k, -Infinity -k for k in (1, 1.0, 1/2, x, 1+I, -1, -1.0, -1/2, -x, -1-I): print k, k / Infinity , k / +Infinity, k / -Infinity for k in (1, 1.0, 1/2, x, -1, -1.0, -1/2, -x): print k, k * Infinity , k * +Infinity, k * -Infinity
there were a small number of things to note
(a) the same answer resulted, regardless of whether (unsigned) Infinity or +Infinity was used. Query: Just to make sure we are getting the results we designed for... Currently +Infinity (or -Infinity) is being returned regardless of whether a signed or unsigned infinity is used. Should (unsigned) Infinity be returned when (unsigned) Infinity is used?
(b) what seems to be an inconsistency occurs when mixing Infinity with complex numbers (same thing holds when we replace Infinity with +Infinity or with -Infinity)
# the following combinations of complex and infinity are ok I + Infinity # +Infinity I - Infinity # -Infinity I / Infinity # 0 # the following crash with Arithmetic Error Infinity / I Infinity * I I * Infinity # isnt I+Infinity (for example) just as meaningful/less as I*Infinity ?
(c) Im curious about the following expressions
x * Infinity -x * Infinity
These return +Infinity
and -Infinity
respectively. But what if x is negative real? (should be opposite answers). The following tries to demonstrate this for two vars (z and x), both declared real in two different ways
sage: var('z',domain='real') z sage: assume(x,'real',x<0,z<0) sage: assumptions() [x is real, x < 0, z < 0] sage: x*+Infinity +Infinity sage: z*+Infinity +Infinity
(is this another ticket "make Infinity work with assumptions/declarations"?)
comment:10 Changed 11 years ago by
Hi Ross,
It's likely that there are inconsistencies in the way infinity is handled by pynac. For instance it definitely doesn't handle interactions with the complex I
well. I implemented support for infinity in pynac to provide a basis for better series expansions and limit computations. However, I didn't have time to actually use it for anything later.
I suggest we keep this issue focused on the conversion problem in the maxima interface so it can be reviewed and merged quickly. You can open separate tickets for the problems in pynac related to infinity. The fact that assumptions are not passed on to pynac should be yet another ticket.
BTW, this article might help decide what behavior we expect w.r.t. arithmetic involving infinity.
comment:11 Changed 11 years ago by
Burcin.
I *did* think it unlikely all these issues would be addressed in one ticket ;-) I guess I was trying to be diligent in reviewing (and Im interested in getting involved in "symbolics" development some time, so the extended exercising of the symbolics module is part of my familiarization process - I even started looking at the code!)
The tests I did above (plus others involving limits) warrant a positive review vote from me. Ill let Karl-Dieter sign it off when he wants, as he has reviewed the code as well.
comment:12 Changed 11 years ago by
I rebased the patch so that it applies cleanly after #7661.
This ticket depends on #7661. Only attachment:trac_8237-maxima_infinity.take3.patch should be applied.
comment:13 Changed 11 years ago by
This looks wonderful to me, but unfortunately I don't have a current build of the alphas for 4.4, so I can't actually test it. I'd give is a positive review if I could!
comment:14 Changed 11 years ago by
- Reviewers changed from Karl-Dieter Crisman, Ross Kyprianou to Karl-Dieter Crisman, Ross Kyprianou, Robert Mařík
- Status changed from needs_review to positive_review
Works for me, long tests passed on 4.4.rc0.
Since also kcrisman wrote "This looks wonderful to me .... I'd give is a positive review if I could!" I think that the ticket deserves positive review.
Positive review. Apply only trac_8237-maxima_infinity.take3.patch
comment:15 Changed 11 years ago by
- Merged in set to sage-4.4.2.alpha0
- Resolution set to fixed
- Status changed from positive_review to closed
reported on sage-devel