Opened 11 years ago
Closed 11 years ago
#5356 closed defect (fixed)
[with patch, positive review] 100r returns a Sage integer in the notebook (but commandline works fine)
Reported by: | jason | Owned by: | boothby |
---|---|---|---|
Priority: | blocker | Milestone: | sage-3.4 |
Component: | notebook | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
type(100r) returns
<type 'sage.rings.integer.Integer'>
in the notebook in 3.3. Similarly, type(1.0r) returns
<type 'sage.rings.real_mpfr.RealLiteral?'>
Both of these examples work fine on the command line (i.e., return python int and float, respectively).
Attachments (1)
Change History (6)
comment:1 Changed 11 years ago by
- Milestone set to sage-3.4
Changed 11 years ago by
comment:2 Changed 11 years ago by
- Summary changed from 100r returns a Sage integer in the notebook (but commandline works fine) to [with patch, needs review] 100r returns a Sage integer in the notebook (but commandline works fine)
comment:3 Changed 11 years ago by
- Summary changed from [with patch, needs review] 100r returns a Sage integer in the notebook (but commandline works fine) to [with patch, positive review] 100r returns a Sage integer in the notebook (but commandline works fine)
comment:4 Changed 11 years ago by
+1 from me.
comment:5 Changed 11 years ago by
- Resolution set to fixed
- Status changed from new to closed
Merged in Sage 3.4.alpha0.
Cheers,
Michael
Note: See
TracTickets for help on using
tickets.
This patch fixes the problem and passes doctests in preparser.py. Someone more familiar with the preparse might look at it, but it's a positive review for me.