#2329: Add interface to PARI's rnfisnorm()
----------------------------------------------------------------------------+
   Reporter:  craigcitro                                                    |   
    Owner:  craigcitro            
       Type:  enhancement                                                   |   
   Status:  needs_review          
   Priority:  blocker                                                       |   
Milestone:  sage-4.7              
  Component:  number fields                                                 |   
 Keywords:  editor_craigcitro pari
     Author:  Craig Citro, Marco Streng, Francis Clarke, Jeroen Demeyer     |   
 Upstream:  N/A                   
   Reviewer:  Nick Alexander, David Loeffler, Jeroen Demeyer, David Kirkby  |   
   Merged:                        
Work_issues:                                                                |  
----------------------------------------------------------------------------+
Changes (by drkirkby):

  * reviewer:  Nick Alexander, David Loeffler, Jeroen Demeyer => Nick
               Alexander, David Loeffler, Jeroen Demeyer,
               David Kirkby


Comment:

 I can confirm that the doctest now passes on a 32-bit build of
 sage-4.7.alpha1 on !OpenSolaris 06/2009 (Intel Xeon chip)

 {{{
 sage -t -long -force_lib
 "devel/sage/sage/rings/number_field/number_field.py"
          [48.8 s]

 ----------------------------------------------------------------------
 All tests passed!
 Total time for all tests: 48.8 seconds
 }}}

 Prior to adding the 3 patches to sage-4.7.alpha1, I got.

 {{{
 File "/export/home/drkirkby/sage-4.7.alpha1/devel/sage-
 main/sage/rings/number_field/number_field.py", line 3035:
     sage: K.selmer_group([K.ideal(2, -a+1), K.ideal(3, a+1), K.ideal(a)],
 3)
 Expected:
     [2, a + 1, a]
 Got:
     [2, a + 1, -a]
 }}}


 I don't know enough about the maths to understand what the patch does, so
 whilst it appears to fix the problems I see in sage-4.7.alpha1, I don't
 feel I should give this a positive review - it needs a mathematician to
 check it too.

 Dave

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