#8327: Implement the universal cyclotomic field, using Zumbroich basis
-----------------------------------------------------+----------------------
Reporter: nthiery | Owner:
Type: enhancement | Status:
needs_review
Priority: major | Milestone: sage-5.6
Component: number fields | Resolution:
Keywords: Cyclotomic field, Zumbroich basis | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Christian Stump, Simon King | Merged in:
Dependencies: #13727, #13728 | Stopgaps:
-----------------------------------------------------+----------------------
Comment (by stumpc5):
Replying to [comment:133 chapoton]:
> Hum, the patch has become so much bigger, and the html preview does no
longer work. It seems that you are now touching many more files. This will
make my work much more difficult. There seem to be a huge number of
whitespace-removal, which is a bit perturbing. I am not happy with this
latest version, indeed..
>
I do agree, I didn't check that carefully. All I did was to add a 5 line
method to {{{rings/number_field/number_field.py}}} that does the
conversion from UCF to CF.
In order to not get trailing white space errors, I removed them all (and
this did in the end modify the complete file, unfortunately). Should I
just leave the white spaces in order to keep the readability of the patch?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8327#comment:135>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
