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

Abfrage der nominatim um Öffnungszeiten dem Land und dem Bundesland anzupassen #35 #37 #40

Merged

Conversation

h8h
Copy link
Contributor

@h8h h8h commented Jun 25, 2018

== Update
Commit bereinigt

Hi,

nachdem ich nun die Woche versucht habe etwas sinnvolles aus der Overpass API rauszuquetschen, denke ich wir tun uns im ersten Schritt leichter direkt die nominatim API abzufragen.

Massenabfragen an diese entstehen nicht, da die Abfrage erst getriggert wird, wenn der Nutzer auf den Marker klickt und auch nur wenn es dort Öffnungszeiten gibt.

Zu meinen verzweifelten Versuchen:
Die Overpass API hat ein Befehl convert mit den man Einträge in Node / Ways / Relation hinzufügen kann, jedoch gehen damit auch Einträge wie Geokoordinaten verloren.

Eine andere Idee wäre für jedes Bundesland die Overpass API zu benutzen. Führt man alles in einem Query aus, fügt Overpass die Daten zusammen und es ist nicht erkenntlich, zu welchem Bundesland die Ergebnisse gehören. Führt man alles in unterschiedlichen Queries aus, wäre es doof, wenn eine davon fehlschlägt, dann fehlt nämlich ein Bundesland und man bekommt es unter Umständen nicht direkt mit. Bei 16 Abfragen innerhalb weniger Minuten könnte auch das Quota anschlagen.

@stefangrotz
Copy link
Member

stefangrotz commented Jun 25, 2018

Hey das klingt großartig. Ich bin morgen den ganzen Tag unterwegs aber hoffe ich kann es mir abends anschauen. Ich packe dann noch einen Satz zur Datenschutzeeklärung hinzu. Außerdem müssen die Änderungen in der Logik des Popups (mehr link typen) im Template nachgezogen werden.

@stefangrotz stefangrotz merged commit 2cfc9ee into CodeforKarlsruhe:new-data-structure Jun 25, 2018
@stefangrotz
Copy link
Member

stefangrotz commented Jun 25, 2018

Vielleicht übernehme ich auch erst mal nur das eine Feature ohne das Template. Das Template wird ja erst sinnvoll sobald wir Mehrsprachigkeit einführen oder wie siehst du das?

EDIT: die Templates sind schon praktisch, ich schaue mal das ich die hoch ziehe. Der link unter der Adresse zerreißt aber regelmäßig das css wenn er zu lang ist, da muss man sich noch was einfallen lassen.

Edit2: der einzige Nachteil an dieser Lösung ist dass eine "jetzt gerade offen" - Funktion nicht möglich ist weil ja immer nur ein Punkt abgefragt werden kann. Aber es ist trotzdem besser als das was es jetzt gibt deswegen würde ich sagen lösen wir ein Problem nach dem Anderen und machen es erst mal so.

@h8h
Copy link
Contributor Author

h8h commented Jun 26, 2018

Guter Punkt, das mit dem "Jetzt Gerade Offen"
Ich denke richtig lösen können wir das vermtl. nur wenn wir uns selber die Planets runterladen und dann direkt die entsprechenden Abfragen machen.

Ich hab wirklich lang rumgesucht und weder die overpass noch die nomatim zeigen alle Daten an die wir brauchen. Erst im Zusammenschluss beider APIs bekommen wir die gewünschten Ergebnisse. Aber paar Tausend Abfragen auf die nomatim API loszutreten und dann die Ergebnisse zusammen zuführen, ist auch etwas unglücklich.

Das Popup.js und das Template, finde ich tatsächlich etwas aufgeräumter.

Für Links wie Facebook und Twitter, gibt es noch die Möglichkeit eigene "Render" zu coden, das müsste ich mir dann auch nochmal anschauen.

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.

2 participants