#9457: power series comparison should use padded_list
------------------------------------------------------------------------------------------------------------------------+
   Reporter:  niles                                                             
                                        |       Owner:  malb      
       Type:  defect                                                            
                                        |      Status:  needs_work
   Priority:  minor                                                             
                                        |   Milestone:  sage-4.5.2
  Component:  commutative algebra                                               
                                        |    Keywords:            
     Author:  niles                                                             
                                        |    Upstream:  N/A       
   Reviewer:                                                                    
                                        |      Merged:            
Work_issues:  Fix bug in sage.schemes.elliptic_curves.sha_tate.Sha.an_padic; 
mention ticket number in commit messages.  |  
------------------------------------------------------------------------------------------------------------------------+
Changes (by SimonKing):

  * work_issues:  Mention ticket number in commit messages => Fix bug in
                  sage.schemes.elliptic_curves.sha_tate.Sha.an_padic;
                  mention ticket number in commit
                  messages.


Comment:

 First, I applied the patches from #9443 and found that all doctests pass.
 Then, I applied the tickets from here, and got one doctest failure:
 {{{
 sage -t  -long devel/sage/sage/schemes/elliptic_curves/sha_tate.py
 Saturation index bound = 265
 WARNING: saturation at primes p > 100 will not be done;
 points may be unsaturated at primes between 100 and index bound
 Failed to saturate MW basis at primes [ ]
 *** saturation possibly incomplete at primes [ ]
 **********************************************************************
 File "/home/king/SAGE/sage-4.4.2/devel/sage-
 main/sage/schemes/elliptic_curves/sha_tate.py", line 393:
     sage: EllipticCurve('53a1').sha().an_padic(5) # rank 1    (long time)
 Exception raised:
     Traceback (most recent call last):
       File "/home/king/SAGE/sage-4.4.2/local/bin/ncadoctest.py", line
 1231, in run_one_test
         self.run_one_example(test, example, filename, compileflags)
       File "/home/king/SAGE/sage-4.4.2/local/bin/sagedoctest.py", line 38,
 in run_one_example
         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
       File "/home/king/SAGE/sage-4.4.2/local/bin/ncadoctest.py", line
 1172, in run_one_example
         compileflags, 1) in test.globs
       File "<doctest __main__.example_4[13]>", line 1, in <module>
         EllipticCurve('53a1').sha().an_padic(Integer(5)) # rank 1    (long
 time)###line 393:
     sage: EllipticCurve('53a1').sha().an_padic(5) # rank 1    (long time)
       File "/home/king/SAGE/sage-4.4.2/local/lib/python/site-
 packages/sage/schemes/elliptic_curves/sha_tate.py", line 567, in an_padic
         raise RuntimeError, "There must be a bug in the supersingular
 routines for the p-adic BSD."
     RuntimeError: There must be a bug in the supersingular routines for
 the p-adic BSD.
 }}}

 So, there will be some bug hunt needed.

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