Opened 11 years ago
Closed 9 years ago
#7965 closed defect (fixed)
quo_rem attribute error (probably easy to fix?)
Reported by: | was | Owned by: | AlexGhitza |
---|---|---|---|
Priority: | major | Milestone: | sage-5.0 |
Component: | basic arithmetic | Keywords: | |
Cc: | mjo | Merged in: | sage-5.0.beta1 |
Authors: | Michael Orlitzky | Reviewers: | Marco Streng, Michael Orlitzky |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #10596 | Stopgaps: |
Description
sage: 5.quo_rem(2/3) --------------------------------------------------------------------------- AttributeError Traceback (most recent call last) /virtual/scratch/wstein/build/sage-4.3.1.rc0/<ipython console> in <module>() /virtual/scratch/wstein/build/sage-4.3.1.rc0/local/lib/python2.6/site-packages/sage/rings/integer.so in sage.rings.integer.Integer.quo_rem (sage/rings/integer.c:16710)() /virtual/scratch/wstein/build/sage-4.3.1.rc0/local/lib/python2.6/site-packages/sage/structure/element.so in sage.structure.element.FieldElement.quo_rem (sage/structure/element.c:15715)() AttributeError: 'sage.rings.rational.Rational' object has no attribute '_parent'
Attachments (1)
Change History (8)
comment:1 Changed 9 years ago by
- Cc mjo added
- Status changed from new to needs_review
comment:2 Changed 9 years ago by
- Reviewers set to Marco Streng
- Status changed from needs_review to positive_review
comment:3 Changed 9 years ago by
- Milestone changed from sage-4.8 to sage-5.0
comment:4 Changed 9 years ago by
- Dependencies set to #10596
- Status changed from positive_review to needs_work
Rebased to #10596, added one test.
comment:5 Changed 9 years ago by
- Status changed from needs_work to needs_review
comment:6 Changed 9 years ago by
- Reviewers changed from Marco Streng to Marco Streng, Michael Orlitzky
- Status changed from needs_review to positive_review
The ticket number is missing from the commit message, but I suppose that's how you want it? The new patch applies and tests cleanly.
comment:7 Changed 9 years ago by
- Merged in set to sage-5.0.beta1
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
This looks fixed, at least for the rationals. I've added a doctest and changed the description of the
other
input which was weird anyway.