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.

Sign up for GitHub

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

Jump to bottom

Openrouteservice #897

Closed
V60GC opened this issue Oct 18, 2018 · 11 comments
Closed

Openrouteservice #897

V60GC opened this issue Oct 18, 2018 · 11 comments
Assignees
@2Abendsegler
Labels
enhancement prio: middle status: completed enhancement, improvement implemented
Milestone
v0.10

Comments

@V60GC
Copy link

V60GC commented Oct 18, 2018

maps.openrouteservice.org

Irgendwie schaffe ich es nicht, mich an den BRouter zu gewöhnen. Währe es möglich den Openrouteservice als Option einzubauen.

Viele Grüße
V60
@CachingFoX
Copy link
Collaborator

Technische spricht nichts dagegen einen weiteren Dienst aufzunehmen.

urlTemplate: 'http://maps.openrouteservice.org/directions?n1={center_latitude}&n2={center_longitude}&n3={zoom}&a=null,null,{waypoints}&b=0&c=0&k1=en-US&k2=km',

Zugklärende Punkte:

  • Routing bei den Home Koordinaten beginnen lassen? (kleiner Implementierungsaufwand)
  • Kann man die Wegpunkt Namen übergeben oder nur die Koordinaten?
  • Unterschiedliche Kartenlayer möglich
  • Was bedeuten die Parameter, b,c,k1,k2?

Ein kurzer Test hat ergeben, dass es nach meine Einschätzung ein geringer Aufwand und geringes Risiko ist.

@Ruko2010, @2Abendsegler Umsetzen?

@2Abendsegler
Copy link
Owner

Das hier hab ich wohl verpaßt. Ja, aus meiner Sicht ok.

@2Abendsegler 2Abendsegler added this to the v0.10 milestone Nov 14, 2019
@2Abendsegler
Copy link
Owner

URL Template

  • n1 Center latitude, wird ergänzt
  • n2 Center longitude, wird ergänzt
  • n3 Zoom, wird ergänzt
  • a Waypoints: latitude, longitude, latitude, longitude, ...
  • b Fußgänger spazieren = 2; Fußgänger wandern = 2b
  • c Präferenz: schnellste = 0; kürzeste = 1; bevorzugt = 2
  • k1 Sprache = en-US, wird ergänzt
  • k2 Maßeinheit = km, wird ergänzt
  • Die Layer können scheinbar nicht übergeben werden.

Beispiel 1:

https://maps.openrouteservice.org/directions?&a=49.74053,8.63237,49.73614,8.64483,49.75067,8.64773,49.76233,8.65881&b=2b&c=0

Beispiel 1: URL Aufbereitung durch Openrouteservice:

https://maps.openrouteservice.org/directions?n1=49.749386&n2=8.647227&n3=14&a=49.74053,8.63237,49.73614,8.64483,49.75067,8.64773,49.76233,8.65881&b=2b&c=0&k1=en-US&k2=km

Beispiel 2: GC1FPN1

https://maps.openrouteservice.org/directions?b=2b&c=0&a=48.13595,11.57055,48.13017,11.58348,47.83808,11.50918,47.65735,11.46245,47.58065,11.44732,47.4177,11.46827,47.36425,11.43677,47.13698,11.635,47.07407,11.69252,47.03743,11.64965,46.99473,11.66033,46.80007,11.74377,46.74472,11.83317,46.58953,11.8297,46.51467,11.82388,46.5018,11.81238,46.4391,11.90778,46.38808,12.03307,46.35407,12.03127,46.2619,12.17622,46.06698,12.24488,45.96992,12.21428,45.73667,12.39128,45.54447,12.59937,45.43377,12.33992,48.13595,11.57055,48.13017,11.58348,47.83808,11.50918,47.65735,11.46245,47.58065,11.44732,47.4177,11.46827,47.36425,11.43677,47.13698,11.635,47.07407,11.69252,47.03743,11.64965,46.99473,11.66033,46.80007,11.74377,46.74472,11.83317,46.58953,11.8297,46.51467,11.82388,46.5018,11.81238,46.4391,11.90778,46.38808,12.03307,46.35407,12.03127,46.2619,12.17622,46.06698,12.24488,45.96992,12.21428,45.73667,12.39128,45.54447,12.59937,45.43377,12.33992

@2Abendsegler 2Abendsegler self-assigned this Nov 14, 2019
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload js #897
d923926
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Merge pull request #1064 from 2Abendsegler/openrouteservice
4459357 
Openrouteservice: Upload js #897
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload defi.js #897
e82a1b8 
DIREKT BIS IN DEN MASTER MERGEN UND SOFORT TESTEN.
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Merge pull request #1065 from 2Abendsegler/openrouteservice
015d7b9 
Openrouteservice: Upload defi.js #897
DIREKT BIS IN DEN MASTER MERGEN UND SOFORT TESTEN.
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload defi.js #897
faac716 
DIREKT BIS IN DEN MASTER MERGEN UND SOFORT TESTEN.
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Merge pull request #1067 from 2Abendsegler/MASTER_defi.js_openroutese…
7cd9440 
…rvice

Openrouteservice: Upload defi.js #897
DIREKT BIS IN DEN MASTER MERGEN UND SOFORT TESTEN.
@2Abendsegler
Copy link
Owner

Folgendes gemerged:

  • Datei "defi.js" von branch "openrouteservice" auf branch "collector".
  • Datei "defi.js" von branch "MASTER_defi.js_openrouteservice" auf branch "master".
  • Datei "defi.js" von branch "master" auf branch "collector".

Damit sind alle notwendigen merges für die Datei "defi.js" erledigt und wir können mit dem geänderten Script "gc_little_helper_II.user.js" vom "collector" ganz normal testen, "defi.js" wird von der branch "master" geladen.
So die Theorie ... 🤣 ... ach Quatsch, ich habs ja schon gestestet. 😁

@2Abendsegler
Copy link
Owner

2Abendsegler commented Nov 14, 2019 •
edited

@V60GC
Hey, kannst du die Änderungen auch mal testen?

Du kannst hier den GClh mit Änderungen installieren und testen.
Anschließend kannst du hier den GClh im Original wieder installieren.

Mögliche Geocaches zum Testen:

@V60GC
Copy link
Author

V60GC commented Nov 14, 2019 via email

@2Abendsegler
Copy link
Owner

Bei Multis mit vielen WPs mit Koords ist natürlich etwas Nacharbeit nötig.

Welche Nacharbeiten sind denn nötig? Wurden Waypoints nicht gefunden?

Ein Bonbon wäre natürlich, wenn es eine Möglichkeit gäbe, die HomeKords mit zuberücksichtigen. ;-)

Man könnte eine Einstellung im Config dafür vorsehen. Direkt beim Openrouteservice Link ist das eher mit viel Aufwand verbunden. Würde es im Config helfen?

2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Maintain changelog #897
040af7b
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload images #897
a8b6fd0
@V60GC
Copy link
Author

V60GC commented Nov 14, 2019 via email •
edited by 2Abendsegler

@V60GC
Copy link
Author

V60GC commented Nov 14, 2019 via email

2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload js #897
a880f5a
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload js #897
7c51137
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Openrouteservice: Upload js #897
a34c11f
2Abendsegler added a commit that referenced this issue Nov 14, 2019
@2Abendsegler
Merge pull request #1069 from 2Abendsegler/openrouteservice
24dea36 
Openrouteservice #897
@2Abendsegler
Copy link
Owner

Hallo Uwe,

Das "Nacharbeiten" war jetzt keine Kritik an der angebotenen Lösung. Das was ich gesehen habe war toll.

Hatte ich auch gar nicht so verstanden. Zudem wäre konstruktive Kritik auch was Gutes. Alles gut. 😊

Beispiel: https://coord.info/GC6A1XJ ... Ich denke, dass es unmöglich sein wird, durch ein Script diesen Weg je in der Richtigen Reihenfolge darzustellen. Das meinte ich mit "Nacharbeiten".

Ganz genau, wenn Wege sich kreuzen oder nahe aneinander vorbeikommen ist es wohl sehr schwierig. Die Routing Plattform sollte das ja eigentlich machen, aber je komplizierter es ist desto mehr Irrwege können entstehen. In der Regel ist wohl bei normalen Multis ... nichts oder kaum etwas nachzuarbeiten.

Die Verwendung der Homekoordinaten als Startpunkt habe ich noch eingebaut.
Außerdem habe ich noch das Medium für die Fortbewegung, also Gehen, Fahrrad, Auto ..., bei den Einstellungen mit aufgenommen.

Hier geht es zu den Einstellungen.
Du kannst hier den GClh mit Änderungen installieren und testen.
Anschließend kannst du hier den GClh im Original wieder installieren.

Wenn das so weit ok ist, dann würden diese Änderungen mit der nächsten Version kommen.

Liebe Grüße
Frank

@2Abendsegler 2Abendsegler added user action status: completed enhancement, improvement implemented and removed status: in progress user action labels Nov 14, 2019
@2Abendsegler
Copy link
Owner

@V60GC
Hey Uwe,
ich schließe das Issue mal.
Falls noch etwas ist, dann bitte melden. Gegebenenfalls machen wir das Issue nochmal auf.
LG Frank

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

@2Abendsegler 2Abendsegler

Labels
enhancement prio: middle status: completed enhancement, improvement implemented
Projects
None yet
Milestone
v0.10
Development

No branches or pull requests
3 participants
@CachingFoX @2Abendsegler @V60GC