Opened 15 years ago
Closed 15 years ago
#1494 closed defect (fixed)
[with patch, with positive review] bug coercing from maximal order of cyclotomic field into cyclotomic field
Reported by: | was | Owned by: | somebody |
---|---|---|---|
Priority: | major | Milestone: | sage-2.9 |
Component: | basic arithmetic | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | Work issues: | ||
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
sage: K.<z> = CyclotomicField(7) sage: O = K.maximal_order() sage: K(O.1) Traceback (most recent call last): ... TypeError: Cannot coerce element into this number field
Attachments (2)
Change History (6)
Changed 15 years ago by
comment:1 Changed 15 years ago by
Changed 15 years ago by
comment:2 Changed 15 years ago by
- Summary changed from bug coercing from maximal order of cyclotomic field into cyclotomic field to [with patch, with positive review] bug coercing from maximal order of cyclotomic field into cyclotomic field
Looks OK to me. The new doctest passes (after applying the trivial patch in trac-1494-fixdoctest.patch).
comment:3 Changed 15 years ago by
I was worried if there was some special reason we were calling _coerce_from_other_number_field here previously, but it looks OK.
This will probably be revised when coercion is flushed throughout the system anyways.
comment:4 Changed 15 years ago by
- Resolution set to fixed
- Status changed from new to closed
Merged in 2.9.rc0.
Note: See
TracTickets for help on using
tickets.
I am wary of this patch, but will look into it more.