Document/Clarify Removal of GraphTraversal Functionality

[michaelweinold]

@cmutel, you moved (some?) graph traversal functionality to the bw_graph_tools package with 273c520. This is not documented in the changelog (or anywhere else except in the commit title). The last changelog entry related to graph traversal simply mentions:

Changed GraphTraversal to AssumedDiagonalGraphTraversal

This means that on the current main branch, there is no graph traversal functionality in the bw2calc package at all.

However, in the latest dev version, there is a bw2calc/graph_traversal.py file . bw_graph_tools is not called anywhere.

Is there some reason to this madness 🙈?

• On the current version of Brightway2 (main), how should one use the GraphTraversal class? Import bw_graph_tools directly?
• On the current version of Brightway25 (dev), how should one use the GraphTraversal class? Call it from bw2calc? What is the role of bw_graph_tools in this case?

[cmutel]

However, in the latest dev version, there is a bw2calc/graph_traversal.py file

You posted a link to 2.0.dev12 - the latest dev release is 2.0.dev16 :)

[michaelweinold]

...oh - it seems that GH can't sort tags properly:

But in any case, the file is also present in the dev16 version.

[cmutel]

You're right, sorry about that. The DEV16 release was on October 22, and the graph traversal was removed on October 26.

Brightway2 is pinned to bw2calc < 2.0.

On the current version of Brightway25 (dev), how should one use the GraphTraversal class? Call it from bw2calc? What is the role of bw_graph_tools in this case?

One should only use bw_graph_tools, this is strictly better in every way than the old graph traversal class.

[michaelweinold]

Ok, thank you for the clarification. Can I therefore:

• remove the bw2calc/graph_traversal.py file from the latest dev version
• and make a corresponding changelog entry

?

[cmutel]

Changelog update in 1dd1132. But the main branch isn't ready for a release yet (yes, yes, this should have been a feature branch and not merged to main...).

So I think for now you will just need to manually use bw_graph_utils instead.

[michaelweinold]

Fine, we'll keep this open until then. Thank you!
I will update the documentation of the bw_graph_tools package in the coming weeks, as I go through the code in detail.