#14990: Implement algebraic closures of finite fields
-------------------------------------------------+-------------------------
       Reporter:  pbruin                         |        Owner:
           Type:  enhancement                    |       Status:
       Priority:  major                          |  needs_review
      Component:  algebra                        |    Milestone:  sage-5.13
       Keywords:  finite field algebraic         |   Resolution:
  closure                                        |    Merged in:
        Authors:  Peter Bruin                    |    Reviewers:
Report Upstream:  N/A                            |  Work issues:
         Branch:                                 |       Commit:
   Dependencies:  #14958, #13214                 |     Stopgaps:
-------------------------------------------------+-------------------------

Comment (by vdelecroix):

 Replying to [comment:15 pbruin]:
 > Replying to [comment:12 vdelecroix]:
 > > Replying to [comment:11 pbruin]:
 > > > Thanks to you all for the comments.  I haven't got a lot of time at
 the moment, but hope to get back to this ticket soon.  In the meantime,
 feel free to continue suggesting improvements or implementing them
 yourselves. 8-)
 > >
 > > Thanks. I am happy to work on this (I already implements the remarks I
 made). Is that ok if I switch to the git workflow ?
 > I haven't done any development with Sage+Git before (only uploaded an
 SSH key and installed Sage using Git), but I'll start trying it out.  Once
 I get some facility in using it, I won't mind if this will become a Git
 ticket.

 Hi Peter,

 I switch to git only recently and it is not easy to go from git to patches
 (the other way around is quite easy using the sage dev scripts). I
 modified several things from your patch, in particular: algebraic_closure
 now works without argument (the name is 'z' by default). I think that I
 will modify the following two problems
  - .is_finite() fails on GF(3).algebraic_closure() (raise a
 NotImplementedError). On the other hand it will be fixed with the new
 category schemes which allows Field().infinite())
  - Zmod(3) does not support algebraic_closure()

 From the last version #15390 (roots of polynomial and eigenvalues) works
 nicely !

 Your work is really nice!

 Cheers,
 Vincent

--
Ticket URL: <http://trac.sagemath.org/ticket/14990#comment:16>
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/groups/opt_out.

Reply via email to