#14990: Implement algebraic closures of finite fields
-------------------------------------+-------------------------------------
Reporter: pbruin | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.3
Component: algebra | Resolution:
Keywords: finite field | Merged in:
algebraic closure | Reviewers:
Authors: Peter Bruin | Work issues:
Report Upstream: N/A | Commit:
Branch: u/pbruin/14990 | 63bc7aa3b23e6d7dc4db54de76c60b24134e7895
Dependencies: #14958, #13214 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by pbruin):
Thanks for your feedback, Vincent! The two new commits should address
your points (1) and (2), respectively. As for (3), I thought about moving
the cache to `AlgebraicClosureFiniteField`, but then we would have to be
very careful with weak references to make sure we don't store every
algebraic closure forever. Another (ugly) solution would be to duplicate
the same default arguments everywhere. I think it is not really worth the
trouble, especially because the following does work:
{{{
sage: GF(5).algebraic_closure('z') is GF(5).algebraic_closure()
True
}}}
I expect it is much more likely that users switch between specifying the
'z' or not than that they switch between specifying keyword arguments or
not.
--
Ticket URL: <http://trac.sagemath.org/ticket/14990#comment:87>
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.
