copy() -> make_twa_graph()
It seems like several people use copy()
(which takes a twa
and return a twa_graph
) when they should use the much more efficient make_twa_graph
(which can take a twa_graph
to return a twa_graph
) instead.
I wonder if we should not deprecate copy()
and provide an overloaded make_twa_graph
that could also take an abstract twa
. The name would make clearer that we are really exploring the entire graph to build a copy, while the current copy
does not necessarily makes it obvious that the result is no longer computed on the fly.