Ticket #7720 (needs_work enhancement)

Opened 4 years ago

Last modified 3 years ago

Digraph.reverse() should be rewritten more efficiently ( not hard )

Reported by: ncohen Owned by: rlm
Priority: major Milestone: sage-5.11
Component: graph theory Keywords:
Cc: rlm Work issues:
Report Upstream: N/A Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description

This function should be rewritten much more efficiently :

First, there should be a way to reverse the arcs "in place" ( without building a copy, by modifying the current graph -- I do not know if the expression used in frech translates in this case ). Such a function, for c_graphs, should be written in Cython and consist in the case of sparse graph in reverting the two copies kept of the graph. In the end, this function should consist in an (optional) copy of the graph (=fast) and a call to the functionr reverting the arcs ( O(1) )

If possible and if deemed useful, the same should be made for NetworkX graphs.

Change History

comment:1 Changed 4 years ago by rlm

In case of a SparseGraph, the SparseGraphBackend has self._cg and self._cg_rev. They can merely be swapped. In the case of a NetworkX graph, I believe their digraphs have succ and pred dictionaries storing the adjacency information. These could also be swapped. For DenseGraphs, this is equivalent to switching between row ordering and column ordering in bitsets, which is inevitably costly.

comment:2 Changed 3 years ago by jason

  • Type changed from defect to enhancement

comment:3 Changed 3 years ago by ncohen

  • Status changed from new to needs_work
Note: See TracTickets for help on using tickets.