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

changeset does not close after 22h

Open mfrasca opened this issue 3 years ago • 8 comments

Describe the bug changeset https://www.openstreetmap.org/changeset/114241653 was opened at 2021-11-26T00:16:43Z, and at the time of writing is still open.

Expected behavior an idea could be to close the changeset as soon as the next item goes at more than 200m away from the previous element.

could it be something being changed while processing older bug #327?

mfrasca avatar Nov 26 '21 22:11 mfrasca

the same is happening with https://www.openstreetmap.org/changeset/114281829, opened shortly after 00:00Z (that's about 12 hours ago at the moment of writing), still open, cumulating contributions from all over the world.

mfrasca avatar Nov 27 '21 12:11 mfrasca

Thanks for this issue! We are in the middle of rewriting the process how Wheelmap syncs OSM data. One goal is to create more manageable changesets. I‘m sorry I can’t tell you an exact date when all this will be finished. Will keep you posted :)

opyh avatar Dec 07 '21 21:12 opyh

hi. if you mark all your commits with #329, before you push them to this public repository, we can follow what you're doing and who knows even help you in the process. I had a look at some of your links, but did not understand how it fits with the interaction with OSM.

mfrasca avatar Dec 08 '21 13:12 mfrasca

am I guessing correctly, you have reduced the span from 24h to 1h? this does reduce the symptoms, I hope your team won't consider the issue solved. :wink:

mfrasca avatar Dec 19 '21 18:12 mfrasca

No worries :)

In an upcoming version, changing a place's wheelchair accessibility on Wheelmap will synchronously create one changeset. No asynchronous queue. While this will fail more often, people will get early feedback if something does not work.

Background: We have experienced that because edits do not get propagated to OSM directly, people knowing how OSM works are afraid to lose their edits. Data should simply not be visibly out of sync between Wheelmap and OSM.

We can tag changesets with meta-information like a random (anonymous) browser session ID, and mapathon information, if existent. This way, edits that have something in common get more transparent.

opyh avatar Feb 08 '22 18:02 opyh

While this will fail more often, people will get early feedback if something does not work.

Eric Raymond's The Art of Unix Programming § Rule of Repair: Repair what you can — but when you must fail, fail noisily and as soon as possible

Lee-Carre avatar Feb 10 '22 16:02 Lee-Carre

The PR mentioned above will create atomic changesets, as described above. It's not in there yet, but will be soon. There will be an intermediate beta deployment to test this feature.

opyh avatar Dec 05 '22 13:12 opyh

Tach Herr Zappe :-)

Könntest Du mich aus den Wheelmap-Banachrichtigungen rausnehmen?

Viele Grüße & Vielen Dank, Rainer


Interaction / Visual Designer

Mobil: +49 157 525 00 922

Framefield GmbH Paul-Robeson-Straße 46 Telefon: +49-30-60 052 742 D-10439 Berlin Internet: www.framefield.com

Sitz der Gesellschaft: Berlin; USt-IdNr.: DE283994303 Handelsregister: Amtsgericht Charlottenburg - HRB 142573 B

Am 05.12.2022 um 14:21 schrieb Sebastian Felix Zappe @.***>:

The PR mentioned above will create atomic changesets, as described above. It's not in there yet, but will be soon. There will be an intermediate beta deployment to test this feature.

— Reply to this email directly, view it on GitHub https://github.com/sozialhelden/wheelmap-frontend/issues/329#issuecomment-1337328187, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEBDUH65AHHAQQJX3C2MUU3WLXT4NANCNFSM5I3NZARA. You are receiving this because you are subscribed to this thread.

nero-still avatar Dec 05 '22 14:12 nero-still

We're currently working on this in https://github.com/sozialhelden/wheelmap-frontend/pull/372. You will be able to create changesets using your OSM account.

opyh avatar Mar 16 '23 10:03 opyh

Closing changesets instantly seems to work in a test. https://www.openstreetmap.org/changeset/133760058

It will still take a while to polish the new app for a public release.

opyh avatar Mar 16 '23 18:03 opyh

this came up again in the OSM generic Telegram group. image

mfrasca avatar May 20 '23 05:05 mfrasca

https://github.com/sozialhelden/wheelmap-frontend/issues/327#issuecomment-1587123854

opyh avatar Jun 12 '23 11:06 opyh

The beta deployment that fixes this issue is online (blog post, beta.wheelmap.org). Waiting for community feedback, then this will go into production.

opyh avatar Sep 13 '23 13:09 opyh

This seems to work reliably. Please reopen if you see something that I might have missed.

opyh avatar Nov 29 '23 11:11 opyh