Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NetworKit shortest path benchmark #2

Open
angriman opened this issue Feb 15, 2020 · 1 comment
Open

NetworKit shortest path benchmark #2

angriman opened this issue Feb 15, 2020 · 1 comment

Comments

@angriman
Copy link
Contributor

Hi, I've noticed that the NetworKit shortest-path benchmark is executed with

distance.BFS(g, node_index).run()

However, with this API NetworKit will also store all shortest paths from node_index to all the other nodes, which implies a significant memory and time overhead. This behavior is a bit counterintuitive, and should be better documented, one does not expect BFS to store by default all shortest paths.
Since the other tools only compute the shortest distances, for a fair comparison you should run this:

distance.BFS(g, node_index, storePaths=False).run()
@timlrx
Copy link
Owner

timlrx commented Apr 16, 2020

Hi, thanks for pointing this out and apologies that I did not get back earlier. Will use your suggested version in the upcoming re-run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants