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

NSI tags applied verbatim when combining features #6951

Closed
1ec5 opened this issue Oct 20, 2019 · 4 comments
Closed

NSI tags applied verbatim when combining features #6951

1ec5 opened this issue Oct 20, 2019 · 4 comments
Labels
preset An issue with an OpenStreetMap preset or tag

Comments

@1ec5
Copy link
Collaborator

1ec5 commented Oct 20, 2019

When combining two features, one of which is associated with a preset based on name-suggestion-index, the combined feature prefers the NSI tags over any tags that were found on one of the two original features.

Steps to reproduce:

  1. Draw a building area.
  2. Draw a point and tag it with the “United States Post Office” preset.
  3. Rename the point to “Smallville Post Office”, because most U.S. post offices are named after the towns they’re in.
  4. Combine the point into the building area.

The combined feature is named “United States Post Office”. It should have retained the name “Smallville Post Office” given in step 3.

Alternative steps to reproduce:

  1. Draw a hospital area.
  2. Draw a point and tag it with the “VA Medical Center” preset.
  3. Rename the point to “Metropolis VA Medical Center”.
  4. Combine the point into the building area.

The combined feature is named “VA Medical Center” rather than “Metropolis VA Medical Center”.

Perhaps this is happening due to the same mechanism described in #6920 (comment). Would it help if NSI were more explicit about which brands can have individually named locations?

@1ec5 1ec5 added the preset An issue with an OpenStreetMap preset or tag label Oct 20, 2019
@bhousel
Copy link
Member

bhousel commented Oct 20, 2019

Map POIs as points separately from building footprints and this isn't a problem.

@bhousel bhousel closed this as completed Oct 20, 2019
@1ec5
Copy link
Collaborator Author

1ec5 commented Oct 20, 2019

This isn’t just about buildings. Did you see the alternative steps to reproduce, which is about hospital grounds (amenity=hospital areas)?

@bhousel
Copy link
Member

bhousel commented Oct 21, 2019

I think that's #6561
People want to be able to have the name tag not applied strictly for some brands.

@1ec5
Copy link
Collaborator Author

1ec5 commented Oct 21, 2019

Yes, a fix for #6561 would probably fix the merging issue “for free”. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preset An issue with an OpenStreetMap preset or tag
Projects
None yet
Development

No branches or pull requests

2 participants