MarcusWolschon / osmeditor4android

Vespucci is a OpenStreetMap editor for Android

Home Page:http://vespucci.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

add a generic descriptive changeset comment if mapper did not add any changeset comment.

Colonel-Dixie opened this issue · comments

Hello,
If the user did not add a changeset comment, let the app generate a generic descriptive changeset comment for the changeset, like the one used with other phone apps, for example:

Created 3 pharmacies, a clothes shop, and 8 other objects; Updated 3 hotels and a hospital

This will improve and save us a lot of time wasted investigating and trying to understand what the user did and edit especially with the uncommunicative users.

This proposal came to my mind as there is a user who refuses to add changeset comments after many blocks by DWG and comments, and Andy from DWG told me

As you've no doubt seen, nothing I've tried within the last couple of years as worked.

So if the DWG can't help, this feature will fix this issue.

Thank you and Best Regards.

I understand the sentiment, but Vespucci already moans as much as reasonably possible if comments and source tags are missing and automatically adding text will just make it -easier- for users that don't want to add something meaningful.

There are number of 2nd order issues that would need to be thought about too, for example that even a moderately large changeset would run out of space in the tags.

@SomeoneElseOSM any comments?

if comments and source tags are missing and automatically adding text will just make it -easier- for users that don't want to add something meaningful.

I agree with this statement. Maybe give them a warning before uploading and asking for a changeset comment, and if they ignore it will be that generic descriptive one. Or maybe prevent users from uploading without changeset comment like Browser and PC Java editors? Even though it might not guarantee the changeset comment to be accurate and descriptive, users might add just "edit" or a gibberish text, but it will still make them write something hopefully useful.

I support this, with the change that a changeset comment should be generated and inserted by default, which users can edit or replace if desired.

Most of my changeset comments can be automated - they take the form <area or route surveyed> - <additions>; <changes>; <deletions> (similar to OsmAnd's generated changeset comments). Automating it would save me time (I may have to correct the location, but it would be a significant reduction in time and effort spent), and prevent errors (I may miscount how many addresses or buildings I added, if I added a lot of them).

Additionally, applications like OsmAnd, EveryDoor, and StreetComplete do away with user-created changeset comments entirely, reducing friction in contributing. This makes them more accessible to casual mappers. While some complexity will remain in a general-purpose editor like Vespucci, it can certainly close this one gap.

Additionally, applications like OsmAnd, EveryDoor, and StreetComplete do away with user-created changeset comments entirely,

This is further one of the most criticized features of these apps, and is of no use at all. It amounts to automatically showing the finger to all other mappers.

In my experience as a Vespucci user (and I've re-checked the latest version just now) Vespucci gets the "amount of badgering" about right. Most people add good changeset comments, and people who just don't want to, won't. Having Vespucci make up something based on content adds relatively little. If it persuades some people to "go with the generic comment instead", that wouldn't be a good thing.

Additionally, applications like OsmAnd, EveryDoor, and StreetComplete do away with user-created changeset comments entirely,

This is further one of the most criticized features of these apps, and is of no use at all. It amounts to automatically showing the finger to all other mappers.

Sure it does, but in my opinion, all editors should implement this in order to avoid this "finger" issue. Writing a comment, which is already just a generic reiteration of what you added/removed/modified, for a fairly big changeset from scratch is nothing but a waste of time, at least in the 21st century.

So the question is - why not work towards on having it automatically initiated for every editor? Only then we could solve this "finger" issue without being in the 90s.

https://www.openstreetmap.org/changeset/143570791 as a prime example of auto-generated changeset comments ;)

@projjalm, I find your comment about changeset comments being the stuff of yesterday somewhat disrespectful. If you want to contribute to OSM in a meaningful way, spending ten seconds on writing a good changeset comment is not too much to ask. See https://wiki.openstreetmap.org/wiki/Good_changeset_comments for why that makes sense. Auto-generated comments are useless. If someone cannot be bothered to write a changeset comment, I'd rather that person did not contribute to OSM at all. Someone who finds it "too 90s" to tell their fellow mappers in a human-readable way what they did can set up their own project in which the data is fully generated by AI. No need for human editors at all. Too 90s.

Auto-generated comments are useless. If someone cannot be bothered to write a changeset comment, I'd rather that person did not contribute to OSM at all

That might be your opinion, and I understand emotions are running high, but I find such over-encompassing generalizations is bad reasoning, and not very welcoming OSM behaviour either (to say the least).

So, you would really be OK with rejecting all StreetComplete and EveryDoor contributions as work of their users is "useless because auto-generated changeset comments could (sometimes) be better written manually"? I hold your opinions in high regards usually, but here I hope you can see how such wording can be seen as quite disrespectful too (even if you won't consider challenging that claim)

I'd much prefer if the discussion can be taken in constructive way. I'd too like the option to update changeset comments in some cases too (e.g. see my comment Zverik/every_door#68 (comment)) but in reality, auto-generated comments of on-the-ground-mappers are in 95% of the cases more than good enough, and I don't think I could write much better manual comments myself.

If you @woodpeck can write better comments then me, please see example at https://community.openstreetmap.org/t/rfc-deprecate-hashtag-only-changeset-comments/105770/44 and instruct me how I could write better comment for that example changeset, which would greatly improve the experience of my fellow mappers.

Or, if you concede that such changeset comments (as in that example) are fine and cannot be noticeably improved by manual editing, then you could revoke your blanket condemnation of all auto-generated changeset comments and accept that in some cases (not all of them!) auto-generated comments are just fine and in fact save users from needlessly wasting their time. Then we can try to determine what cases are fine and what are not, and have some common ground on discussions on what should be improved (and what is just fine as is)

Dear @woodpeck , Thank you for your comment, although I always greatly appreciated and respected your opinions and have never disagreed with them, I disagree with you here on the "Auto-generated comments are useless.", I will explain why, and I will give you an example of how it might help in OSM,

For example, see: https://www.openstreetmap.org/user/iran%20map%2028/history
All of his changesets are "(no comment)" according to https://hdyc.neis-one.org/?iran%20map%2028 he has 18,614 changesets and (6.5%) with comments and only 1.5% are unique comments, which means over 17,405 changesets are "(no comment)".

You mentioned a changeset as an example of how auto-generated comment is not useful, but "Created an unknown object" at least will let me know that the user "Created" an object, not like the user I mentioned as an example above who is editing existing objects in OSM, and you have to waste time and investigate the edit to understand what the edits are.

I did not want to name an app name, or a user to put them under a spotlight, as this is not my the point of this issue. Before creating and posting this issue here, I spoke with Andy about the user above and he told me "As you've no doubt seen, nothing I've tried within the last couple of years as worked." Andy blocked him many many times before for that, see: https://www.openstreetmap.org/user/iran%20map%2028/blocks

That is why we need at least an auto-generated changeset comment feature or at least prevent the user from uploading a changeset without comment like the iD editor, and then maybe we will have another issue when people add nonhelpful words like this case: https://www.openstreetmap.org/user/majidred/history when the user only say "fix" "add" "delete" and do not say what he fixed, delete or add, but that would be in issue for another day if we did it.

The rest of your comment here I agree with 100%. The time I wasted on investigating changesets without comments would've been spent on improving and mapping OSM, and that's why changeset comment is an important and must-have feature.

Thank you.

  • I agree with @woodpeck that it is very useful to have an option to add extra manual comments in cases where it would be adding useful information to other mappers (I just disagree that such extra manually added information would be needed in 100% of the cases - or even 50% of them).
  • and I agree with you @Colonel-Dixie that auto-generated comments are useful in many situations, and that it would be very nice to have auto-generated text by default (to be kept as is, or appended onto) in cases where users do not (have the need to) add extra manual comments (as is the case with majority of changesets of on-the-ground mappers where nothing more can be said then already auto-generated "added 10 benches"). I envision it somewhat like JOSM does with "Automatically obtain source from current layers / just once" for source=* tag -- not something users should put on automatic and never change; but it is very useful for 90% of the cases and easy to add too when there is a need.
  • I however am not sure that this particular example by @Colonel-Dixie is a very good example where auto-generated comments would help. After all if the user is malicious (I don't know if that is the case here, but it might be if they are intentionally ignoring all the DWG blocks?), they could put fake information in changeset comments to lower the chances of themselves being spotted (e.g. "added benches" when in fact they were modifying name tags or deleting churches or whatever). In order to get known-good information on changesets (i.e. how many tags were really added/modified/deleted instead of what the changeset author claims has been done) you still need to use some trusted interface like OSMCha instead of blindly trusting what the mapper put in changeset comment.

I'll reiterate that generated changeset comments aren't just for "lazy"/uncooperative editors, but also for assisting "serious" mappers by cutting down on time, effort, and inaccuracies.

I've been thinking about what aspects of changeset comments can be generated. Some that come to mind -

  1. Determining the smallest place within which the changeset was made. This can be placed in the beginning of the comment to textually show the extent of the changeset.

  2. Adding a feature - add [<count>] <preset> e.g. "add 5 bus stops"

  3. Adding details to existing features - add [<count>] <key> e.g. "add 5 names"

  4. When nodes of a way have been added or moved - change [<count>] <feature> geometry e.g. "change primary road geometry" (I would probably change this manually to something more specific, but it's good to have it generated so I don't forget about it in my frequently-large changesets).

  5. Similarly, when the nodes of a way have only been removed - simplify [<count>] <feature> geometry.

  6. When all nodes of a way have been moved the same amount - move <count> <feature>.

  7. delete <feature>

  8. When a new feature has the same keys as a deleted feature - convert <feature> to {node|way|relation}

I'll reiterate that generated changeset comments aren't just for "lazy"/uncooperative editors, but also for assisting "serious" mappers by cutting down on time, effort, and inaccuracies.

Agreed!

I've been thinking about what aspects of changeset comments can be generated. Some that come to mind

One should take into consideration that there is a limit at maximum changeset length (both technical and practical), so there should be some prioritizing among those if not all of them would fit.

What I would suggest is auto-generate that comment and prefill the comment field, and allow user to edit it as they wish (or instead access previous comment via already existing dropdown list -- although unedited comments shouldn't be inserted into that history dropdown, for hopefully obvious reasons).

I would also suggest that when uploading edits, the Properties pane is made the default (instead of Changes pane), or that they get merged together (with comment and source editable fields at the top like e.g. in JOSM), as it often happens to me that I forget to change the pane and enter new (currently manually typed) comment, and thus unintentionally upload changeset with wrong/old comment as it was "hidden" from me in UI.

So, you would really be OK with rejecting all StreetComplete and EveryDoor contributions as work of their users is "useless because auto-generated changeset comments could (sometimes) be better written manually"?

Streetcomplete changeset comments are not created by maschine learning (called AI) but handcrafted with love. But not from the osm user but from the quest author.