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

In the OSRMGeoFilter, if the first euclidean pass did not return any … #277

Merged

Conversation

greenscientist
Copy link
Contributor

…node, don't request data from OSRM

OSRM would return no results as at best it can do the euclidean distance.

Closes: #264

…node, don't request data from OSRM

OSRM would return no results as at best it can do the euclidean distance.

Closes: chairemobilite#264
@greenscientist
Copy link
Contributor Author

Tested locally:
[2023-11-25 10:02:17.190] [info] -- calculating route request -- 0
[2023-11-25 10:02:17.190] [debug] received parameter max_egress_travel_time with value 900
[2023-11-25 10:02:17.190] [debug] received parameter max_access_travel_time with value 900
[2023-11-25 10:02:17.190] [debug] received parameter min_waiting_time with value 180
[2023-11-25 10:02:17.190] [debug] received parameter alternatives with value false
[2023-11-25 10:02:17.190] [debug] received parameter time_of_trip with value 54000
[2023-11-25 10:02:17.190] [debug] received parameter max_transfer_travel_time with value 600
[2023-11-25 10:02:17.190] [debug] received parameter scenario_id with value 68c38af2-91af-49da-8c49-7082f63ea127
[2023-11-25 10:02:17.190] [debug] received parameter destination with value -77.580651,45.477921
[2023-11-25 10:02:17.190] [debug] received parameter max_travel_time with value 10800
[2023-11-25 10:02:17.190] [debug] received parameter time_type with value 0
[2023-11-25 10:02:17.190] [debug] received parameter origin with value -77.603835,45.527467
[2023-11-25 10:02:17.195] [debug] -- reset and preparations -- 5809 microseconds
[2023-11-25 10:02:17.196] [debug] resetting access paths
[2023-11-25 10:02:17.196] [debug] fetching nodes with osrm
[2023-11-25 10:02:17.196] [debug] osrm with host localhost and port 5000
[2023-11-25 10:02:17.198] [debug] There was no node potentially accessible, we did not ask OSRM
[2023-11-25 10:02:17.198] [debug] parsing access footpaths to find min/max access travel times
[2023-11-25 10:02:17.199] [debug] osrm with host localhost and port 5000
[2023-11-25 10:02:17.201] [debug] There was no node potentially accessible, we did not ask OSRM
[2023-11-25 10:02:17.201] [debug] parsing egress footpaths to find min/max egress travel times
[2023-11-25 10:02:17.202] [info] -- route request not found -- 0

@greenscientist greenscientist merged commit 2e1fd30 into chairemobilite:v2c Nov 27, 2023
5 checks passed
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.

OSRM: Do not query the accessible nodes if none are within bird radius
2 participants