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

a new algorithm that returns all possible shortest paths for a source target pair #933

Closed
rok24 opened this issue Jul 18, 2023 · 6 comments
Assignees
Labels
good first issue Good for newcomers
Milestone

Comments

@rok24
Copy link

rok24 commented Jul 18, 2023

At the moment digraph_dijkstra_shortest_paths returns only one shortest path for a source-target pair. It would be nice to see all the possible shortest paths with the same length for a source-target pair.
Currently networkx has this functionality in all_shortest_paths function and it returns all possible shortest paths.

@mtreinish mtreinish added the good first issue Good for newcomers label Jul 24, 2023
@mtreinish mtreinish added this to the 0.14.0 milestone Jul 24, 2023
@prpr2770
Copy link

prpr2770 commented Aug 1, 2023

@mtreinish I'd like to work on this problem. Please let me know how I may proceed, as I'm new to this project.

@mtreinish
Copy link
Member

Sure thing, I've assigned you the issue. The contributing guide: https://github.com/Qiskit/rustworkx/blob/main/CONTRIBUTING.md has some general guidelines for writing PRs. Besides that in this case you'll want to add 2 new functions to https://github.com/Qiskit/rustworkx/blob/main/src/shortest_path/mod.rs that computes all the shortest paths between two nodes for both graphs and digraphs

@prpr2770
Copy link

@mtreinish Thank you for assigning this and sharing the resources. I shall send you an update as soon as its completed.

@IvanIsCoding IvanIsCoding modified the milestones: 0.14.0, 0.15.0 Oct 18, 2023
@lucasvanmol
Copy link
Contributor

Hi @mtreinish, since this hasn't seen much activity recently, could I take this over? I have something that works, but would like some input on the best way to implement it.

@mtreinish
Copy link
Member

@lucasvanmol sure, this has been idle for > 2 months so I've reassigned the issue to you. I'll try to take a look at your WIP PR soon.

@IvanIsCoding
Copy link
Collaborator

Closed by #1017

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

No branches or pull requests

5 participants