#10025: sloane_sequence(111776)[2][:36] != sloane.A111776.list(36)
----------------------------------+-----------------------------------------
   Reporter:  donmorrison         |       Owner:  sage-combinat  
       Type:  defect              |      Status:  needs_review   
   Priority:  minor               |   Milestone:  sage-4.6.1     
  Component:  combinatorics       |    Keywords:  combinat sloane
     Author:  Yann Laigle-Chapuy  |    Upstream:  N/A            
   Reviewer:                      |      Merged:                 
Work_issues:                      |  
----------------------------------+-----------------------------------------

Comment(by ylchapuy):

 Hi,
 Here, the primary goal is to correct the fact that sloane.A111776 is in
 fact the sequence A109814.
 I also added a test to check that be sure this doesn't happen again, which
 test every sequences against the online version.
 With respect to this, I'm happy with the changes made by Nathann and would
 give a positive review.

 I do agree that it's a bad thing not to test the input for correctness,
 but it's not the point of this ticket. David, please feel free to open
 another one to adress this issue. I also let you make the decision about
 giving a positive review or not, as I'm a little bit biased as an author.

 cheers,
    Yann

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