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

Update graphhopper to 0.11.0-pre1 #1

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@develar

develar commented Jun 5, 2018

0.10.1 is released, no need to use snapshot.

(I simply edit file online, didn't check locally).

@develar

This comment has been minimized.

develar commented Jun 9, 2018

I have removed req.setAlgorithm(Parameters.Algorithms.DIJKSTRA_BI) because it is default value for CH and it will lead to issues if landmarks is used instead of CH (I didn't test landmarks yet).

I have checked — works for me (I have built local apk).

@develar develar changed the title from Update graphhopper to 0.10.1 to Update graphhopper to 0.11.0-pre1 Jun 9, 2018

@menion

This comment has been minimized.

Member

menion commented Jun 10, 2018

Thank you develar for a contribution!
I was forced due to some changes, use SNAPSHOT version of GH API, but to be true, I would rather stay with stable versions if not necessary.
So I've made your changes manually together with update of TARGET_SDK_VERSION, support libs & Locus API. Thanks!

@menion menion closed this Jun 10, 2018

@develar

This comment has been minimized.

develar commented Jun 25, 2018

JFYI I cannot yet prepare PR about landmarks support because of graphhopper/graphhopper#1401

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment