#19227: Graphs: DFS and broken distance-parameter
-------------------------------------+-------------------------------------
       Reporter:  jmantysalo         |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.9
      Component:  graph theory       |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jmantysalo/graphs__dfs_and_broken_distance_parameter|  
89b7490bec7e28f1c63bb81603eeab69fefb4e65
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jmantysalo):

 * status:  new => needs_review
 * commit:   => 89b7490bec7e28f1c63bb81603eeab69fefb4e65


Old description:

> This bug is visible at least on normal 64-bit Linux machine running self-
> compiled Sage. Might be architechture-dependent.
>
> {{{
> G = DiGraph({1:[2,10],2:[3],3:[4],4:[5],5:[6],10:[4]})
> list(G.depth_first_search(1, distance=3))
> }}}
>
> This should output also `5` as it is three jumps from 1: `1->10->4->5`.
> It is not outputted because `4` is already marked as seen vertex for
> `1->2->3->4`.
>
> I found this when doing #19197 and there is already some discussion about
> it. I suspect that this might be used  in other places too, and that must
> be checked first.
>
> Also note that #19223 might have collision as the fix may touch almost
> same lines.

New description:

 This bug is visible at least on normal 64-bit Linux machine running self-
 compiled Sage. Might be architechture-dependent.

 {{{
 G = DiGraph({1:[2,10],2:[3],3:[4],4:[5],5:[6],10:[4]})
 list(G.depth_first_search(1, distance=3))
 }}}

 This should output also `5` as it is three jumps from 1: `1->10->4->5`. It
 is not outputted because `4` is already marked as seen vertex for
 `1->2->3->4`.

--

Comment:

 I had to delete and modify some examples. Otherwise this just gives
 deprecation warning.
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=89b7490bec7e28f1c63bb81603eeab69fefb4e65
 89b7490]||{{{Deprecation of distance-parameter on
 depth_first_search().}}}||

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