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

Incorrect Successor/Predecessor Mapping Causing Path Planning Failures #77

Open
Novadgaf opened this issue Jul 6, 2023 · 2 comments
Open
Labels
bug Something isn't working

Comments

@Novadgaf
Copy link

Novadgaf commented Jul 6, 2023

I've been experiencing an issue regarding the routing graph within the libOpenDRIVE project, where the mapping of successors and predecessors does not seem to be functioning as expected. This misalignment in the routing graph is causing my path planning to fail.

To reproduce this issue, you can load the Carla Map Town 10HD on https://odrviewer.io/ and attempt to plot a route from road 90 to road 6. It's currently not possible to find this route, even though physically, the path is feasible. Instead, the tool indicates road 90 has only road 7 as a successor, leaving out road 6.

My understanding is that the correct behavior should involve a valid path from road 90 to road 6, indicating the correct successor/predecessor relations.

This problem likely resides in the part of the code responsible for constructing the routing graph. I would appreciate any insights into why this problem may be happening or tips on how to correct the issue in the routing graph.

@pageldev pageldev added the bug Something isn't working label Sep 16, 2023
@markomiz
Copy link

markomiz commented Oct 5, 2023

I could be misunderstading the code or the openDRIVE standard but maybe this is due to the routing graph assuming that if B is the successor of A then A is the predecessor of B. However, if the roads point toward each other (i.e. the reflines end in the same place) then the behaviour should be that A is also the successor of B.

void RoutingGraph::add_edge(const RoutingGraphEdge& edge)

@markomiz
Copy link

markomiz commented Nov 9, 2023

@Novadgaf take a look at my fork - https://github.com/markomiz/libOpenDRIVE/tree/feature/fix_routes I fixed the issue and created a PR to the original

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants