#8124: Selmer groups for number fields
---------------------------------+------------------------------------------
   Reporter:  rlm                |       Owner:  davidloeffler  
       Type:  enhancement        |      Status:  positive_review
   Priority:  major              |   Milestone:  sage-4.3.2     
  Component:  number fields      |    Keywords:                 
     Author:  Robert Miller      |    Upstream:  N/A            
   Reviewer:  cremona, wuthrich  |      Merged:                 
Work_issues:                     |  
---------------------------------+------------------------------------------
Changes (by cremona):

  * status:  needs_review => positive_review
  * reviewer:  => cremona, wuthrich


Comment:

 Looks good;  tests pass.  Building the docs (reference html) revealed a
 problem in polynomial_quotient_ring.py, preexisting in the selmer_group
 function:  the macro "\cross" was not recognised.  I could not get
 "\times" to work instead (which is what I normally use for this) but *
 does, so I put that in.

 Regarding Chris's comment:  of course I agree, this should be a proper
 abelian group.  But even making it a semi-functional abelian group of the
 sort used for unit groups and class groups would require quite a bit of
 extra work, since this code finds generators but _not_ the group
 structure.  So I think that should be in another ticket.  (For example, I
 would like to use this function for m=4 and will try to do so.  In that
 case there will in general be generators of order 2 as well as some of
 order 4.)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8124#comment:3>
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.

Reply via email to