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

Geonames records have extra, duplicate, label components #49

Closed
orangejulius opened this issue Jun 3, 2016 · 5 comments
Closed

Geonames records have extra, duplicate, label components #49

orangejulius opened this issue Jun 3, 2016 · 5 comments
Assignees

Comments

@orangejulius
Copy link
Member

orangejulius commented Jun 3, 2016

Our current admin layer lookup mapping tells admin layers to be looked up in all admin layers above that of the record, but also including that of the layers themselves. So, for example, locality records from Geonames will be annotated with a locality record from Who's on First.

It used to be that our label generating system required this. However, now it's actually detrimental: when the name from Geonames does not exactly match the name from WOF, the label contains duplicate sections. Here are some examples:

People's Republic of China, China
City of Detroit, Detroit, MI
New York City, New York, NY, USA

We also appear to not handle localadmin level records very well, because they aren't explicitly configured in the layer mapping. So, for example:

City of New York, Brooklyn, New York, NY USA

@orangejulius orangejulius changed the title Only perform admin lookup for layers _above_ the layer of Geonames records Geonames records have extra, duplicate, label components Jun 8, 2016
@dianashk
Copy link
Contributor

Do we have corresponding acceptance tests for this issue?

@orangejulius
Copy link
Member Author

Good idea, we should add some. Fixing pelias/api#569 would help creating those acceptance tests though

@orangejulius
Copy link
Member Author

We do have acceptance-tests, pelias/acceptance-tests#259 covers these cases

@dianashk
Copy link
Contributor

All fixed except China.

@trescube
Copy link
Contributor

Potential solution: check label generation to see if country can be special-cased to just use name

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

Successfully merging a pull request may close this issue.

3 participants