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

We should add local name:xx tags #2245

Closed
bhousel opened this issue Jan 5, 2019 · 10 comments
Closed

We should add local name:xx tags #2245

bhousel opened this issue Jan 5, 2019 · 10 comments

Comments

@bhousel
Copy link
Member

bhousel commented Jan 5, 2019

In working with KeepRight data a lot lately, I notice that it sometimes flags a warning like this:

It would be nice if this node had an additional tag 'name:XX=Chick-fil-A' where XX shows the language of its name 'Chick-fil-A'.

screenshot 2019-01-05 01 30 33

It seems like the warning is caused if someone adds a name:xx tag and a name and they don't match.

screenshot 2019-01-05 01 32 22

screenshot 2019-01-05 01 32 27

It is only a warning , but I can understand why it would be nice in this case to add a name:en tag, so that people know what language name is in or where it came from.

We have a lot of names like this in the index. Our Contributing guidelines even encourage it:

  "amenity/fast_food|マクドナルド": {         // Identifier like "key/value|name"
    "count": 1445,
    "countryCodes": ["jp"],                 // Optional `countryCodes` - array of countries where this entry is used
    "tags": {
      "amenity": "fast_food",
      "brand": "マクドナルド",                // `brand` - Brand name in the local language (Japanese)
      "brand:en": "McDonald's",             // `brand:en` - For non-English brands, tag the English version too
      "brand:wikidata": "Q38076",           // `brand:wikidata` - Same Universal wikidata identifier
      "brand:wikipedia": "ja:マクドナルド",   // `brand:wikipedia` - Reference to Japanese Wikipedia
      "cuisine": "burger",
      "name": "マクドナルド",                 // `name` - Display name, also in the local language (Japanese)
      "name:en": "McDonald's"               // `name:en` - For non-English names, tag the English version too
     // SHOULD WE ADD "name:ja" TOO??
    }
  },
@kymckay
Copy link
Collaborator

kymckay commented Jan 6, 2019

I can't think of any strong argument against it 👍

@matkoniecz
Copy link
Contributor

from what I see there is no pending issue to remove this complaint in https://github.com/keepright/keepright/issues (I checked as Keep Right is no longer developed).

@matkoniecz
Copy link
Contributor

JOSM also complains (at information level)

For example shop=car, name=ホンダ, name:en=Honda node gets

Missing name:* translation - Missing name:*=ホンダ. Add tag with correct language key. (1)

So I think that it would be OK to add also name:<local>.

@kymckay
Copy link
Collaborator

kymckay commented Jan 24, 2019

The only minor complaint I can think of is that it is basically duplicating a bunch of data 🤔 If the name changes you then change it in two places.

However that's more of an issue with the accepted tagging scheme. Seems more logical to me to just have a language:name=en tag to specify the language of the name tag.

@bhousel
Copy link
Member Author

bhousel commented Jan 24, 2019

It sounds like if both JOSM and KeepRight warn users about it, we should just add the local name tag. Our goal with this project is to help people map stuff the best way possible. 😄

@bhousel bhousel changed the title Should we always add a local name:xx tag when we add a name:en ? We should add a local name:xx tags Jan 24, 2019
@bhousel bhousel changed the title We should add a local name:xx tags We should add local name:xx tags Jan 24, 2019
@sommerluk
Copy link
Contributor

For “Burger King” for example it's the same name in French, German and Englisch. In Germany I would expect rather to find a :de while in France I would expect rather a :fr on an object…

@bhousel
Copy link
Member Author

bhousel commented Jan 25, 2019

For “Burger King” for example it's the same name in French, German and Englisch. In Germany I would expect rather to find a :de while in France I would expect rather a :fr on an object…

For that example, we wouldn't set any name:xx tag.

This issue is saying if we set a name:xx tag, make sure there exists a name:xx tag that matches the name tag.

@kymckay
Copy link
Collaborator

kymckay commented Jan 25, 2019

Oh that makes more sense, I also misunderstood this and thought those warnings wanted name:xx on everything 😆

@bhousel
Copy link
Member Author

bhousel commented Mar 8, 2019

I've updated the CONTRIBUTING doc and also added name:xx and brand:xx tags for situations where the language used is unambiguous. Rule going forward is as stated above:

  • "If we set a name:xx tag, make sure there exists a name:xx tag that matches the name tag."
  • "If we set a brand:xx tag, make sure there exists a brand:xx tag that matches the brand tag."
  • etc..

This should keep people happy! It means there is a lot of work to do in the index to add these tags, but I could add a build warning to assist cleanup.

@matkoniecz
Copy link
Contributor

I could add a build warning to assist cleanup.

That would be very useful! Half of my contributions were prompted by spotting a warning in a build output, it also finds typos.

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

No branches or pull requests

4 participants