#12944: Inconcistencies in the documentation of traveling_salesman_problem
--------------------------------+-------------------------------------------
       Reporter:  ncohen        |         Owner:  jason, ncohen, rlm
           Type:  defect        |        Status:  needs_work        
       Priority:  major         |     Milestone:  sage-5.1          
      Component:  graph theory  |    Resolution:                    
       Keywords:                |   Work issues:                    
Report Upstream:  N/A           |     Reviewers:                    
        Authors:                |     Merged in:                    
   Dependencies:  #12942        |      Stopgaps:                    
--------------------------------+-------------------------------------------
Changes (by kini):

  * status:  needs_review => needs_work


Comment:

 This patch breaks the doctests of the function! So we need to edit those
 doctests. But...

 You said on #10497:
 > Though if I remember correctly the french lines have been removed
 recently in an unrelated patch by David Coudert `:-D`

 Well, this patch is at least not merged in 5.0.rc1, since the French lines
 in the docstring are still there - if I am to edit the doctests, I should
 know where that patch is, so I don't conflict with it. But because the
 patch is unrelated, I can't search for it - do you remember which patch it
 is? (Here is an example of why one should keep unrelated changes out of
 patches! :P)

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