#18152: Universal Cyclotomic Field implementation using libgap
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.7
Component: number fields | Resolution:
Keywords: | Merged in:
Authors: Vincent Delecroix | Reviewers: Jean-Philippe Labbé
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/vdelecroix/18152 | 28c23ce280985112ea43f0bea36a76952a64ff8e
Dependencies: #18153 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by nthiery):
The main argument I could see for keeping the code around is as follow:
The speed balance may evolve in the future, typically if free modules in
Sage get optimized, or if new use cases emerge. Of course we could always
revive the code then, but it would probably have rotten in the mean time,
and we may have completely forgotten about it. It also can be used for
testing purposes for comparing two independent implementations of UCF.
Those are not super strong arguments.
The main argument for not keeping it is that we would not want is wasting
time maintaining it.
A sensible approach might be to write a brief comment at the beginning of
the file / class stating something like: "at this point this code is not
used much (see ...). If maintaining it becomes a bother in the future,
it's ok to discard it".
--
Ticket URL: <http://trac.sagemath.org/ticket/18152#comment:41>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
