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

Value autocomplete wrong behavior #6441

Closed
Bibi56 opened this issue May 26, 2019 · 12 comments
Closed

Value autocomplete wrong behavior #6441

Bibi56 opened this issue May 26, 2019 · 12 comments
Labels
wontfix-low-impact Doesn't improve the user experience enough to spend time on

Comments

@Bibi56
Copy link

Bibi56 commented May 26, 2019

Entering stan for the key camp_site proposes standard (correct) but also standart (wrong).
For German-speaking people the second may sound good, it is not.
Where the possible values are limited, only valid values (present on the Wiki, accepted proposals...) should be proposed.
Elsewhere, ill formed values (like my-proposed-feature where my_proposed_feature is proposed) should be eliminated.

@BjornRasmussen
Copy link
Contributor

iD currently uses taginfo, which stores all commonly used keys. This means that if a key has a lot of use, the key will be suggested to the user typing in the raw tag editor, regardless of whether or not the key is approved

@Bibi56
Copy link
Author

Bibi56 commented May 26, 2019

I'm speaking about values, not keys.
I check the occurrence of standard (82) and standart (2).
2 (two) usages worldwide is not what I would call "a lot of use".
And, even without checking with wiki recommended values, the difference between the 2 usages (with similar values) is huge.

@bhousel
Copy link
Member

bhousel commented May 27, 2019

Thanks for checking @Bibi56 - if there are just 2 instances worldwide and it's an obvious mistake, just change them.

@bhousel bhousel closed this as completed May 27, 2019
@bhousel bhousel added the wontfix-low-impact Doesn't improve the user experience enough to spend time on label May 27, 2019
@Bibi56
Copy link
Author

Bibi56 commented May 27, 2019

That's exactly my point: because they are almost not used, they shouldn't get proposed!
BTW, I changed the two invalid values.

@Marc-marc-marc
Copy link
Contributor

2 suggestions to improve the quality of proposals made through taginfo:

  • hide values that have less than x% usage without a wiki page, this should hide typing errors
  • hide depreciated values

@BjornRasmussen
Copy link
Contributor

That could turn into controversy about which tags are considered to be depreciated and which tags aren't. They best solution is to just remove depreciated values from the database so iD doesn't suggest them.

@quincylvania
Copy link
Collaborator

iD does hide values from the suggestion that are on the deprecated list (#6084).

I actually don't mind low-usage tags getting suggested. Maybe someone is mapping with a great new tag they created. More importantly, there are plenty of "accepted" tags with relatively low usage. Some people might be confused or angry if these didn't appear as expected.

@Marc-marc-marc
Copy link
Contributor

That could turn into controversy about which tags are considered to be depreciated

this would be surprising since the wiki page listing the deprecated tags is seen by a large number of contributors.

just remove depreciated values from the database

it's obviously always better/&needed to fix the database but it's a misunderstanding of osm to believe that all deprecated tags can quickly disappear.
moreover, the list of deprecated tags does not prevent typos.
today if someone makes a typo, iD will propose the typo in spite of itself since it exists in taginfo.
that's why I proposed a simple way to improve the quality of iD suggestions while avoiding that iD have to create a black list by hand, which would be an endless job

@Bibi56
Copy link
Author

Bibi56 commented May 30, 2019

For low usage values like standart, a soundex comparison with common values could help

@Bibi56
Copy link
Author

Bibi56 commented Jun 8, 2019

Yesterday a contributor used dispersed as value because it was proposed in the drop-down list for camp_site. At this time nobody was using camp_site=dispersed. So some values must come from outside of taginfo.

@quincylvania
Copy link
Collaborator

Yesterday a contributor used dispersed as value because it was proposed in the drop-down list for camp_site.

@Bibi56 How can you tell? iD doesn't suggest this for me (though maybe someone removed the usage).

Screen Shot 2019-06-10 at 8 34 19 AM

At this time nobody was using camp_site=dispersed. So some values must come from outside of taginfo.

Nope, the values in the "All tags" editor come only from taginfo. Check the code. Some fields in the "All fields" section do have predefined suggestions, but there isn't a field for camp_site.

@Bibi56
Copy link
Author

Bibi56 commented Jun 10, 2019

@quincylvania Yes the creator of the node camp_site=dispersed removed the tag.
I can tell because I wrote him "Hi, I've seen you are using non standard values.
dispersed is not a possible value for camp_site. basic is probably the correct value here.". At the time he added the value, taginfo was unaware of this value. One day later it gave the value (that's how I found this usage). The user answered me "You are correct about no “dispersed” camp_site, nothing in the Wiki, but then why does it come up as possible value when you scroll down in values?" and for the changeset he was using ID. Therefore my deduction. Today I don't see "dispersed" as proposed value either which confirms what you're saying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix-low-impact Doesn't improve the user experience enough to spend time on
Projects
None yet
Development

No branches or pull requests

5 participants