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

Fixes two problems with Substation #1338

Merged
merged 1 commit into from
Apr 20, 2013

Conversation

Vanuan
Copy link
Contributor

@Vanuan Vanuan commented Apr 20, 2013

http://wiki.openstreetmap.org/wiki/Tag:power%3Dsub_station

  1. substation -> sub_station.
  2. Substations are usually buildings (doesn't apply to points unfortunately).

1. substation -> sub_station.
2. Substations are usually  buildings (doesn't apply to points unfortunately).
@jfirebaugh
Copy link
Member

Not convinced about building=yes. The wiki shows an outdoor area, which is what most substations I've seen look like, and according to taginfo only 30% of power=sub_station features are building=yes. I believe the correct tagging will in most cases be to tag the overall area as power=sub_station and individual buildings on premise with building=yes.

@jfirebaugh jfirebaugh merged commit 4d3060f into openstreetmap:master Apr 20, 2013
@jfirebaugh
Copy link
Member

Merged the substation -> sub_station correction, thanks!

@Vanuan
Copy link
Contributor Author

Vanuan commented Apr 20, 2013

See my comments in #1214. Without "building=yes", this preset is useless, i.e.:

  1. it doesn't refrect currently mapped features with "power=sub_station", "building"="yes" tags as a Substation
  2. "create a Substation -> deselect -> click substation" workflow doesn't work
  3. clicking an existing (correctly mapped) feature with "building=yes" or "area"="yes" and setting a Substation preset removes "building"="yes" tag.

What's the use of this preset then?

@Vanuan
Copy link
Contributor Author

Vanuan commented Apr 20, 2013

The wiki shows an outdoor area, which is what most substations I've seen look like

What most substations I've seen look like this: http://wiki.openstreetmap.org/w/images/thumb/d/d3/Trafostation_Alter_Hellweg_IMGP4722_osm.jpg/300px-Trafostation_Alter_Hellweg_IMGP4722_osm.jpg

We're not talking here how to correctly map those, but how to adapt the editor to currently mapped state. Probably, area=yes should be applied to those that are not buildings automatically. I don't know. I only know that there is a bug.

jfirebaugh added a commit that referenced this pull request Apr 20, 2013
@jfirebaugh
Copy link
Member

Yes, there was another bug caused by iD not recognizing anything tagged with power=* as an area. Try http://www.openstreetmap.us/iD/master/ -- 1 and 2 should be fixed. Switching from the Building preset to the Substation preset will still remove building=yes -- that's largely by design because presets are currently exclusive. We don't yet have a concept of an auxiliary or secondary preset.

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 this pull request may close these issues.

2 participants