#6312: [with patches, needs review] optional doctest failure -- galois_group
-------------------------------+--------------------------------------------
 Reporter:  was                |       Owner:  tbd       
     Type:  defect             |      Status:  new       
 Priority:  major              |   Milestone:  sage-4.0.3
Component:  optional packages  |    Keywords:            
 Reviewer:                     |      Author:            
   Merged:                     |  
-------------------------------+--------------------------------------------

Comment(by fwclarke):

 I now suspect (but can't check) that the second failure does not lie with
 the use of magma but arises because the type was undefined.
 The doctest comment anyway
 indicates that the type should have been set to 'gap', so I've fixed
 that.  But this needs to be checked by someone with magma.

 It turns out that leaving both `type` and `names` as `None` always causes
 a failure when the field
 is not Galois since no name is then provided for a generator of the
 Galois closure.  This has been fixed in the new patch, and a doctest
 adjusted appropriately.

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