wheelmap-classic icon indicating copy to clipboard operation
wheelmap-classic copied to clipboard

Split your OSM-changesets by wheelmap_visitor into much smaller areas

Open holgerd opened this issue 10 years ago • 9 comments
trafficstars

An OSM-user writes:

Please split your changesets into much smaller areas. Very large global changesets are very difficult for local mappers to review and find what has actually been changed in their area of local knowledge.

holgerd avatar Oct 02 '15 13:10 holgerd

Actually, we had @momolinus working in this algorithm, but it was not implemented yet. Please have a look here:

https://github.com/momolinus/WmOptimisation/wiki/Ergebnis-v2

christoph-buente avatar Oct 02 '15 15:10 christoph-buente

http://wiki.openstreetmap.org/wiki/User:Roald-linus was working on this at the OSM hack weekend in Berlin 2015: https://www.buero20.org/fertig-gehackt-openstreetmap-hackweekend-im-rueckblick/

holgerd avatar Oct 03 '15 07:10 holgerd

@holgerd it is the same person ;)

christoph-buente avatar Oct 04 '15 08:10 christoph-buente

Hallo Ihr zwei, im Prinzip habe ich die Optimierung ja schon Ende 2014 abschließend getestet (gut Ding will Weile haben).

Einfach ausgedrückt:

  • das Wheelmap-Backend muss die geografische Größe berechnen und speichern, die Änderungen (= neuer Wert für wheelchair- Attribut) erzeugen würden
  • würde eine Änderung eine gegebene Größe überschreiten, fordert das Wheelmap-Backend ein neues Changeset an
  • das Backend sollte alle offenen Changesets speichern (also die ID) und für jede neue Änderung prüfen, ob sie in ein Changeset passt, das das Backend schon hat
  • der "Preis" ist eine höhere Anzahl von Changesets

"Simuliert" habe ich das schon, die Ergebnisse habe ich hier aufgeschrieben: https://github.com/momolinus/WmOptimisation/wiki/Ergebnis-v1.1#ergebnisse-f%C3%BCr-area-guard-00073

Beim letzten Hackweekend wollte ich den Algorithmus so beschreiben, dass ein anderer Programmierer ihn in einer anderen Sprache implementieren kann. Am Ende des Hackweekends habe ich den Kollegen meinen Text vorgestellt (hier: https://github.com/momolinus/WmOptimisation/blob/v2.1/Wheelmap%20Optimisation/ressources/Algorithmus.odt). Dabei wurde ich auf einen syntaktischen Fehler im Programmablaufplan hingewiesen. Die Korrektur wollte ich auch gleich in den Code übertragen. Leider schlugen danach meine JUnit-Tests fehl und das Hackweekend war zu Ende. Also durchhalten :relaxed: - nein, ich denke wenn jemand den Algorithmus implementieren will, geht das sofort. Am besten wäre es, ich treffe mich mit dem/der Programmierer/in. Laut Raul pflegt das Wheelmap-Backend inzwischen ja ein Unternehmen, oder?

Viele Grüße Marcus

momolinus avatar Oct 04 '15 09:10 momolinus

Thank you @momolinus for the information and detailed work on this problem.

@skade Are you able to work with this?

lennerd avatar Oct 05 '15 08:10 lennerd

Is this why my change https://www.openstreetmap.org/changeset/34766069 is showing such a huge area when I only changed a couple (?) of attributes?

ghost avatar Oct 21 '15 02:10 ghost

Hello @voltagex ,

no, your changeset is so huge, because changsets "geographic size" encloses all changed attributes. It would be the same, if you use JOSM as OpenStreetMap editor.

If some anonymous wheelmap users change attributes in different countries (or even continents) within one hour (or day) their changeset encloses all their changes and becomes much greater. Such a changeset could enclose a few countries. But we are working on this problem.

greetings

momolinus avatar Oct 21 '15 04:10 momolinus

Update from @momolinus

He found some issues with his former solution and updated it here: https://github.com/momolinus/WmOptimisation/wiki

holgerd avatar Jun 14 '16 12:06 holgerd

Since this repo is now dead, see: wheelmap-frontend #327: “Changeset bounding box area”.

Lee-Carre avatar Oct 25 '21 19:10 Lee-Carre