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

geometry_simplify does not work for > 2 locations #457

Closed
hhblaze opened this issue Mar 12, 2019 · 2 comments
Closed

geometry_simplify does not work for > 2 locations #457

hhblaze opened this issue Mar 12, 2019 · 2 comments
Assignees
Labels
awaiting release ⏳ bug 🐞 Erroneous behavior of the backend

Comments

@hhblaze
Copy link

hhblaze commented Mar 12, 2019

Here's what I did

Getting directions

Getting directions with geometry_simplify=True


Here's what I got

{"error":{"message":"You cannot change an immutable PointList"},"info":{"engine":{"version":"5.0","build_date":"2019-02-28T08:13:19Z"},"timestamp":1552385578459}}

Here's what I was expecting

the nice result

Here's what I think could be improved

With geometry_simplify=False it works, please, check the function

@rabidllama
Copy link
Contributor

Hi @hhblaze - we have recently been updating the api to get things like the simplify_geometry to work properly, and as such the geometry_simplify=true parameter cannot be used on routes that have more than one segment (in the Graphhopper core library I believe this is due to the simplify changing the geometry itself which can mess up aspects relating to via points). I have added an error message that says this is the case, so this should appear on the servers in the next few days. We will get it added to the documentation as well

@rabidllama rabidllama self-assigned this Mar 12, 2019
@rabidllama rabidllama added bug 🐞 Erroneous behavior of the backend awaiting release ⏳ labels Mar 12, 2019
@hhblaze
Copy link
Author

hhblaze commented Mar 12, 2019

Thank you!

@hhblaze hhblaze closed this as completed Mar 12, 2019
@nilsnolde nilsnolde changed the title Getting directions parameter is not working anymore geometry_simplify does not work for > 2 locations Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting release ⏳ bug 🐞 Erroneous behavior of the backend
Projects
None yet
Development

No branches or pull requests

2 participants