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

Added a new k disjoint shortest path algorithm for sparse graphs #797

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bkrogh
Copy link

@bkrogh bkrogh commented Jun 26, 2019

Added a new k disjoint shortest path algorithm which is faster for the cases with sparse graphs and where the k disjoint paths from source to sink are similar to the shortest path tree rooted in the source vertex

This addresses issues: #796

Added a new algorithm which is a slightly altered version of the existing Bhandari implementation.

I welcome any comments that may improve the code quality (and improve chances of this getting merged)


bbk@blipsystems.com and others added 3 commits June 26, 2019 15:46
faster for the cases with sparse graphs and where the k disjoint paths
from source to sink are similar to the shortest path tree rooted in
the source vertex
@Toptachamann
Copy link
Collaborator

Hi,

This looks interesting. Cloud you please include an experimental running time comparison of both implementations. There're some examples of how we implement benchmark tests under jgrapht/jgrapht-core/java/test/test/org/jgrapht/perf/....

Please, also try to find out the cases when the previous implementation is more efficient. We have to state this as clear as possible in the Java doc so that there's no doubt for the users which one to use.

@jkinable
Copy link
Collaborator

What's the status of this PR?

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

Successfully merging this pull request may close these issues.

None yet

3 participants