#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 davidloeffler):
I concur with fwclarke's diagnosis -- the traceback indicates that magma's
not getting called, it's using the new code I wrote (wrapping Pari). This
is just a hangover from ticket #5159: I changed the arguments to
galois_group, and didn't check I'd fixed the optional doctests (it took
about five attempts just to get the non-optional ones to work!). And it
looks like nobody else checked them either until now.
The patches look spot on based on browsing the code, but I am at a
conference at the moment so don't have time to download and test them (and
I don't have Magma either).
David
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6312#comment:3>
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
-~----------~----~----~----~------~----~------~--~---
