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

Allow "layer" tags alone to resolve building crossing issues #6367

Closed
wants to merge 3 commits into from

Conversation

quincylvania
Copy link
Collaborator

Closes #5924.

This lets instances of buildings crossing highway/railways/waterways to be disambiguated by simply setting different layer tags. The covered tag is ignored altogether.

Leaving this as a draft since I'm still not 100% sure this is the desired behavior.

Screen Shot 2019-05-15 at 12 13 05 PM

This also makes makes the Layer editor visible in the Structure field even without a structure tag if a "layer" tag is present

Don't allow the "covered" tag to resolve building crossing issues
Show the layer editor in the Structure field even without a structure tag if a "layer" tag is present
@BjornRasmussen
Copy link
Contributor

Thanks for this!! The JOSM validator currently works like this, so this will be nice to avoid confusion about the validator

@quincylvania quincylvania added the validation An issue with the validation or Q/A code label May 15, 2019
@bhousel
Copy link
Member

bhousel commented May 15, 2019

I'm wary of this approach, because I still think the ground has an implied layer 0, and features below it may not render.

@BjornRasmussen
Copy link
Contributor

I agree with @bhousel - we need to make sure that, for example, a service road under a roof doesn't get the tag layer=-1

@cserpell
Copy link

Please make this through, this issue is driving me nuts with some new editors that do not understand that adding the layer=1 to a building=roof (specially in gas stations) is enough to make all service ways below to be ok.

@kymckay
Copy link
Collaborator

kymckay commented May 28, 2019

@cserpell According to the layer tag wiki page this is not true

When ways are passing on different levels apply layer=* only to the way which also has the bridge/tunnel attribute. Only ways with one of the tags/attributes tunnel=, bridge=, highway=steps, highway=elevator, covered=* should be tagged with the layer tag, similar for railways and waterways.

I'm not sure where steps and elevators come into it, but this matches my understanding of the layer tag. The feature still must be tagged appropriately

@matkoniecz
Copy link
Contributor

Fortunately negative layer does not mean that something is underground, explicit location=underground should be used to tag this.

@quincylvania
Copy link
Collaborator Author

Does this approach (not requiring covered=yes) sound okay if iD wouldn't assign negative layers?

@bhousel
Copy link
Member

bhousel commented Jul 19, 2019

If you want to hard-code rules for roofs, go ahead.

@quincylvania
Copy link
Collaborator Author

If you want to hard-code rules for roofs, go ahead.

I don't particularly. But I think if any building has layer=1 with a crossing road then it's implied that the building covers the road and iD shouldn't flag it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation An issue with the validation or Q/A code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drive-Through crosses Roof warning
6 participants