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

Wrong intersection_node ID in edge based graph factory #6166

Closed
samDB-gv opened this issue Dec 3, 2021 · 2 comments
Closed

Wrong intersection_node ID in edge based graph factory #6166

samDB-gv opened this issue Dec 3, 2021 · 2 comments

Comments

@samDB-gv
Copy link

samDB-gv commented Dec 3, 2021

Hello,

When running osrm-contract with osrm v5.26 (to take into account #6020 ), logs gives me something like :

[warn] Turn is a u turn but not turning to the first connected edge of the intersection. Node ID: 2539866, OSM link: http://www.openstreetmap.org/?zoom=18&mlat=48.970389&mlon=2.282888

But this node id was deleted 15 years ago ^^
Since intersection_node is derived from NodeID, is this related to the fact that NodeID is still defined as uint32_t in typedefs.hpp ?

Another hint comes from graph_contract_adaptors.hpp, where the warning "[warn] Edge weight large -> 1084267 : 1649465 -> 1649466" lead to deleted nodes.

Thanks for your reply,

@mjjbell
Copy link
Member

mjjbell commented Dec 3, 2021

The log message is a bit misleading, it's referring to the internal OSRM Node ID and hoping you use the link to figure out the OSM details. We know the OSM ID at this point in the processing, it will just require a bit of plumbing to make it available to the log message.

Since intersection_node is derived from NodeID, is this related to the fact that NodeID is still defined as uint32_t in typedefs.hpp ?

No, whilst OSM Node IDs now exceed UINT32_MAX, OSRM is still able to enumerate all the nodes it needs for creating
routing networks within 32 bits (although for large planet size OSM extracts, it's starting to get reasonably close).

@samDB-gv
Copy link
Author

samDB-gv commented Dec 6, 2021

oh yes ok, I see. Thanks for your explainations !

@samDB-gv samDB-gv closed this as completed Dec 6, 2021
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