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

Isochrone center cannot be set as "destination" #171

Closed
pcopissa opened this issue Jun 28, 2022 · 1 comment · Fixed by #191
Closed

Isochrone center cannot be set as "destination" #171

pcopissa opened this issue Jun 28, 2022 · 1 comment · Fixed by #191
Assignees

Comments

@pcopissa
Copy link

pcopissa commented Jun 28, 2022

According to ORS isochropne API (https://openrouteservice.org/dev/#/api-docs/v2/isochrones/{profile}/post) it is possible to use the optional field location_type to specify that the points in the locations field are destinations (that is, the output isochrone is the set of points from which we can reach the location in less than X minutes or meters). However the default is start (that is, the output is the set of points reachable in less than X minutes, starting from the location). There is no GUI widget (or any other way ?) to specify that the locations should be destinations.

Expectation is that there should be a way to specify which type of location is desired.

@koebi koebi added this to the ORS API feature completeness milestone Jun 28, 2022
@koebi
Copy link
Collaborator

koebi commented Jun 28, 2022

Hey,

yes, the QGIS plugin is unfortunately not feature complete in regard to the API, and it is not something we're currently working on.
We'll happily review a pull request implementing this, though :)

Best regards

@merydian merydian linked a pull request Nov 8, 2023 that will close this issue
@koebi koebi closed this as completed in #191 Dec 8, 2023
koebi added a commit that referenced this issue Dec 8, 2023
fixes #171 
---------

Co-authored-by: Jakob Schnell <Jakob.Schnell@heigit.org>
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 a pull request may close this issue.

3 participants