#9375: more documentation & doctests for BalancedTree, BarbellGraph,
BubbleSortGraph, BullGraph, ChvatalGraph
-------------------------------+--------------------------------------------
   Reporter:  mvngu            |       Owner:  jason, ncohen, rlm
       Type:  enhancement      |      Status:  needs_review      
   Priority:  major            |   Milestone:  sage-4.5          
  Component:  graph theory     |    Keywords:                    
     Author:  Minh Van Nguyen  |    Upstream:  N/A               
   Reviewer:                   |      Merged:                    
Work_issues:                   |  
-------------------------------+--------------------------------------------

Comment(by ncohen):

 Excellent ! You can set this one to "positive review" immediately after
 #9375 :-)

 One unimportant detail, though... You used subgraph_search in your
 doctests, the follwing way :

 {{{
 sage: s_K = g.subgraph_search(K_n1, induced=True)
 sage: s_P = g.subgraph_search(P_n2, induced=True)
 sage: K_n1.is_isomorphic(s_K)
 }}}

 Well, subgraph_search should of course *always* return subgraphs
 isomorphic to S_k. Actually, the order of the vertices should even be the
 same, but when it finds nothing, it returns a None, which is_isomorphic
 may not like... It's not really a problem in this case, as this would
 report a doctest failure anyway :-)

 Nathann

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