OpenSfM icon indicating copy to clipboard operation
OpenSfM copied to clipboard

networkx graphiz api change causes error in `plot_matches --graph`

Open vik748 opened this issue 5 years ago • 2 comments

Running plot_matches --graph fails with

  File "./OpenSfM/bin/plot_matches", line 64, in <module>
    nx.draw_graphviz(

This is probably due to API change with networkx. If we replace that line with nx.drawing.nx_agraph.graphviz_layout(graph) it works but I can't figure out how to get the edge_color argument correctly.

vik748 avatar Aug 11 '20 01:08 vik748

Hi @vik748 , thank you for reporting this bug, we're going to address it, feel free to make a PR if you already have a fix.

YanNoun avatar Mar 15 '21 10:03 YanNoun

Current implementation https://github.com/mapillary/OpenSfM/commit/ca219bd863dd11656401ba83f7ed68236f64b761 doesn't use networkx anymore.

jonasdlindner avatar May 06 '22 14:05 jonasdlindner