#18839: Boost Dominator Tree
-------------------------------------+-------------------------------------
       Reporter:  borassi            |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.8
      Component:  graph theory       |   Resolution:
       Keywords:  Dominator tree,    |    Merged in:
  Boost                              |    Reviewers:
        Authors:  Michele Borassi    |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  a6f3f9accc844816a3fa720eb8abd154c4a3a805
  u/borassi/boost_dominator_tree     |     Stopgaps:
   Dependencies:  #18811, #18564     |
-------------------------------------+-------------------------------------

Comment (by dcoudert):

 Install OK, docbuild OK, doc looks good.

 However, I have questions on the expected behavior:
 - with 1-vertex graph:
 {{{
 sage: G = graphs.PathGraph(1)
 sage: G.dominator_tree(0)
 {0: None}
 sage: G.dominator_tree(0, return_dict=False)
 Graph on 0 vertices
 }}}
 - With disconnected graphs
 {{{
 sage: G = 2 * graphs.PathGraph(1)
 sage: G.dominator_tree(0)
 {0: None, 1: None}
 sage: G = 2 * graphs.PathGraph(2)
 sage: G.dominator_tree(0)
 {0: None, 1: 0, 2: None, 3: None}
 }}}
 If this is effectively what we expect, then I will finalize the review.

--
Ticket URL: <http://trac.sagemath.org/ticket/18839#comment:10>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to