#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 ncohen):

 Hello !!!

 Here is another patch to add to yours... The doctests did not pass, as the
 sloane_sequence method always outputs "Searching Sloane's online
 database...". I tried to find ways around, then thought the best was to
 just let it forward a verbose parameter to find_sloane... Anyway, I made
 this modification, and I also moved most of the documentation written in
 the __init__ method of class A109814 just after the class definition. This
 way, it appears in the documentation when it is built.

 The same should be done for the other methods, as currently very few of
 them appear :
 http://www.sagemath.org/doc/reference/sage/combinat/sloane_functions.html

 Only my patch needs done to be reviewed before this ticket can be set as
 positively reviewed ! `:-)`

 Nathann

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