Opened 9 years ago
Closed 9 years ago
#14045 closed defect (fixed)
Comparison of float('inf') and infinity
Reported by: | tscrim | Owned by: | tscrim |
---|---|---|---|
Priority: | major | Milestone: | sage-5.7 |
Component: | coercion | Keywords: | infinity, coercion |
Cc: | Merged in: | sage-5.7.beta3 | |
Authors: | Travis Scrimshaw | Reviewers: | Robert Bradshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Currently infinity and float do not compare properly:
sage: oo > float('inf') True sage: oo == float('+inf') False
because of
sage: InfinityRing(float('inf')) A positive finite number
Sage-devel topic: http://groups.google.com/forum/?fromgroups=#!topic/sage-devel/GMv525C0YcY
Attachments (1)
Change History (8)
comment:1 Changed 9 years ago by
- Status changed from new to needs_review
comment:2 Changed 9 years ago by
- Description modified (diff)
comment:3 Changed 9 years ago by
- Status changed from needs_review to positive_review
comment:4 Changed 9 years ago by
- Reviewers set to Robert Bradshaw
- Status changed from positive_review to needs_work
How was this patch file created? It is missing a "User" field, you should create a $HOME/.hgrc
file as explained in http://sagemath.org/doc/developer/producing_patches.html and export the patch using hg export tip
.
Changed 9 years ago by
comment:5 Changed 9 years ago by
- Status changed from needs_work to positive_review
Forgot to export. Sorry. Fixed.
comment:6 Changed 9 years ago by
Sorry I missed that, I was too busy looking at the patch's content :-P.
comment:7 Changed 9 years ago by
- Merged in set to sage-5.7.beta3
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Makes a lot of sense to me, thanks.