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

Add categories to the "What kind of building is this?" Quest #2177

Closed
RiffLord opened this issue Oct 18, 2020 · 43 comments
Closed

Add categories to the "What kind of building is this?" Quest #2177

RiffLord opened this issue Oct 18, 2020 · 43 comments

Comments

@RiffLord
Copy link
Contributor

Use case

Some areas have many abandoned buildings and/or ruins which can't be adequately classified by the categories available in the What kind of building is this? quest. I believe it would be useful to be able to mark these buildings as abandoned.

On a related note, I believe the quest would also benefit from having a category for historical buildings which could encompass used, unused, public or private buildings of historical relevance which otherwise wouldn't fit into any of the currently available categories. While many historical buildings are used for purposes which can be classified with the options available (Civic, Religious or Commercial buildings) there are some which don't fall into these categories and might otherwise remain unmarked.

Proposed Solution

Add an Abandoned building/Ruin and a Historical building option to the Other category in the options for the quest.

Something like this (the icons were repurposed from those available just as a placeholder):

Untitled

@matkoniecz
Copy link
Member

Note that abandoned bunker should still be answered as "bunker", abandoned shed as "shed" and so on.

Historic buildings answer - which tag would be used in this case?

@RiffLord
Copy link
Contributor Author

Regarding generic abandoned buildings, what about other categories which might cause confusion?
For example, in the area where I live there is the ruin of a restaurant. If I were to classify it as a Shop per the app's guidelines, wouldn't it show up as a result along with other commercial activities for people using OSM to find services in the area?

As for the historic buildings, I assume that simply using a Historic tag would be too vague perhaps?
In that case what about creating a separate Historic category, including tags such as Castle, Fort, Watchtower, Observation Post, etc?

The reasoning behind this is that while all over the world buildings such as these are often museums or tourist attractions and can be tagged appropriately through the app, what about the similar places which don't have this function?
How do we tag those historic buildings which remain officially unused but accessible to people?

@westnordost
Copy link
Member

It would be possible to tag something like abandoned=yes or historic=yes. These tags are even well in use.

@RiffLord
Copy link
Contributor Author

So these characteristics should be specified by editing tags after the fact on OpenStreetMaps rather than adding options to the quest in the StreetComplete app?

@westnordost
Copy link
Member

westnordost commented Oct 19, 2020 via email

@westnordost
Copy link
Member

For example, in the area where I live there is the ruin of a restaurant. If I were to classify it as a Shop per the app's guidelines, wouldn't it show up as a result along with other commercial activities for people using OSM to find services in the area?

No, because this quest is just about the building type, ot the amenity within.

@RiffLord
Copy link
Contributor Author

RiffLord commented Oct 19, 2020

As for the historic buildings, I assume that simply using a Historic tag would be too vague perhaps?
In that case what about creating a separate Historic category, including tags such as Castle, Fort, Watchtower, Observation Post, etc?

I understand. In this case I'd say it makes little sense to add an Abandoned tag to this quest, maybe it can be done elsewhere. What about Historic? Is it too generic? Would the quest benefit from a new category for these buildings with tags as suggested above?

@westnordost
Copy link
Member

Did you just cite yourself?

Anyway, maybe a historic category would make sense. But just "historic building" would be fine too in my opinion. What exactly do you have in mind for possible selections in this category?

@RiffLord
Copy link
Contributor Author

Yeah, as a reminder of the ideas I had for the selections which could be a part of the possible historic category.
However if you feel that a generic Historic building tag is enough, I've already added it to my fork of the app, in the "Other" section. I could test to see if everything works as expected.
Is there anything I should do or be aware of before making a pull request?

@westnordost
Copy link
Member

I've already added it to my fork of the app

Nice!

Is there anything I should do or be aware of before making a pull request?

Well, a historic building should be tagged with building=* + historic=building (or =yes). An abandoned building with building=*+abandoned=yes. Ruins with ruins=yes`. Do you want to include all three options?
Any idea for fitting icons?

@RiffLord
Copy link
Contributor Author

RiffLord commented Oct 19, 2020

I'd be glad to work on including all of the mentioned options. I'll take some time to study the code further and find out how to approach tagging these buildings. As of this moment, if I tag a building as Historic from the app, the result on OSM is building=historic, so it classifies the building as such rather than adding a historic=yes tag:

tag

I think a historic building could use the same icon as Civic and Government:

Untitled

@westnordost
Copy link
Member

Don't worry about the icons, I will draw them after your PR is merged. I was just asking for ideas.

Tagging it correctly will not be much effort, just look at how it is done with the man_made=storage_tank.

@rhhsm
Copy link

rhhsm commented Oct 22, 2020

Lots of disused and abandoned buildings where I'm mapping, but I don't think SC is suitable for tagging them. I think a building should first of all be tagged with the type it was built for, and then an abandoned or disused tag can be added if it's no longer used. If we allow a building to be tagged as abandoned using SC, it would mean no quests would appear any more asking if it was a house, church, office, etc. isn't it? I think that is worse than not tagging it. A separate quest asking of every building if it might be disused or abandoned would be a lot of spam. I can imagine it could be an additional question to the building type quest, but again the answer will almost always be no (I hope). I think the historic tag has the same problem: it's a secondary characteristic of a building. Any building should first of all be tagged for what it was built for (not what it is used for), and then maybe for its historic value.

@westnordost
Copy link
Member

westnordost commented Oct 23, 2020

@rhhsm I think SC is very much suitable for tagging them. SC is a tool for people doing ground survey. Where else other than on a survey is it possible to determine whether a building is ruined or abandoned?

But I understand your point of view: the property (abandoned or not) is separate from building=* and thus should be recorded separately. But since it is not appropriate to ask for every single building if it is abandoned or not, your verdict is, that this is not a property that should be recorded with StreetComplete.

I see it from a difference perspective, from the perspective of a surveyor (a user of the app): If the building is abandoned or even ruined, or it is just some kind of historic building, it is quite impossible for the anyone on-site to find out what this building was created for. It's just "some building like any other". Because it is not well possible to answer this, the quest is not asked for abandoned buildings, ruined buildings, historic buildings, military buildings, buildings that are (tourist) attractions, buildings tagged with power etc.

So, from a surveyor perspective, it very much makes sense to answer "(I can't answer this because) the building is abandoned". Without this option, the user would not be able to give an answer to this case, but the app should enable the user to do so. Since the quest just generally asks for "what kind of building this is?", it makes sense to put this answer option amongst all the others instead of into the special "other answers...." menu.

Regarding your point that tagging ruins=yes (etc.) leads to that the quest is not shown (for other users) any more: Yes, that's right, but this is exactly the point. If someone on site cannot determine what kind of building it is because it is abandoned, then it makes sense to not ask every passing user again about it. Just like for opening_hours:signed=no. That doesn't mean that anyone else with knowledge from another source cannot set the building type (or f.e. the opening hours) later, just not with StreetComplete.

@rhhsm
Copy link

rhhsm commented Oct 23, 2020

I'm in favour of having an answer option "Can't say because the building is so damaged that its function cannot be determined", but then the tag to add is ruins:building=yes, as "abandoned" means that "Such features will still have some physical form reflective of their former use visible in the landscape." https://wiki.openstreetmap.org/wiki/Key:abandoned: If their former use is still visible, it should be tagged for it.

@westnordost
Copy link
Member

If their former use is still visible, it should be tagged for it.

Sure, but why wouldn't people do that even if there is such an "abandoned" option? What building type would you select if you see an apartment building which seems to be deserted? Which one is the defining property of a building - it being an apartment house or it being some kind of deserted building is in the eye of the beholder. The "openstreetmap tags" categorization can't be forced on users if it does not coincide with how in reality the building is perceived.

@rhhsm
Copy link

rhhsm commented Oct 24, 2020

I'd definitely tag it as an apartment building with SC, and then use Vespucci or iD to add a disused or abandoned tag. I believe that the consensus on OSM is that buildings should first of all be tagged with what they were built for, and then in addition, if applicable, for what they are used for and for their lifecycle status. I don't think SC should tempt mappers to deviate from the OSM consensus.

@westnordost
Copy link
Member

StreetComplete should enable mappers to map what they see.

@mnalis
Copy link
Member

mnalis commented Oct 25, 2020

@rhhsm I agree with the part is USAGE differs, eg. something was built as a church, and later transformed to pub, would be building=church and building:use=retail. However, while current use (building:use) is almost always easily known, original purpose is in many cases hard to know on-site (although there are exceptions like church or warehouse where it is much easier to correctly guess).

However, building=ruins / abandoned=yes and similar tags are often special, as it might not even be possible to discern original purpose (especially for very old, or razed to the ground in war and never rebuilt - as is often the case in parts of my country).

And even it was possible, something that is basically a pile of collapsed bricks should IMHO not be tagged just as building=yes or building=house, with hope that someone will later remember to mark it as ruins/abandoned in desktop editor.

I would think that marking it as building=ruins or similar would be preferable in such situations.

@rhhsm
Copy link

rhhsm commented Oct 26, 2020

I think there's a lot of confusion here about the meaning of abandoned: Please read https://wiki.openstreetmap.org/wiki/Key:abandoned: The defining difference between an abandoned building and a ruined one, is that the former purpose of an abandoned building can still be recognised, and that of a ruined building not. If a house in a very bad state of repair can still be recognised as a house, it should be tagged abandoned:building=house. If it cannot be recognised, it should be tagged ruins:building=yes
Also note the taginfo counts here https://wiki.openstreetmap.org/wiki/Lifecycle_prefix (ruins: and ruined: are seldom used).

@westnordost
Copy link
Member

westnordost commented Oct 26, 2020 via email

@rhhsm
Copy link

rhhsm commented Oct 27, 2020

Hmm. the Nikolaikirche is indeed a tricky one. From the Wikipedia article I understand that only the tower is still standing, is being maintained and is used as a memorial, while the rubble of the rest of the church has been removed. I'd tag it with man_made=tower, tower:type=bell_tower, building=yes (because of guidance here https://wiki.openstreetmap.org/wiki/Tag:tower:type%3Dbell_tower ) and historic=monument (to reflect its current use). Just after it was bombed it was a ruin, but since it's being maintained since then, it's not a ruin any more. It's being used, so not abandoned. And the rest of the church is gone, so not building=church.

You wrote above that "StreetComplete should enable mappers to map what they see." I fully agree: "Map what's on the ground" is one of the good practices of OSM. But there are many borderline cases where different mappers may choose different options, and in order to have some consistency in mapping, there are further guidelines, formed by consensus, that I think SC should follow. An example where SC follows an exception to "Map what's on the ground" is abbreviations on street signs https://wiki.openstreetmap.org/wiki/Names#Abbreviation_.28don.27t_do_it.29

I used the lifecycle prefix because the simple tag is discouraged https://wiki.openstreetmap.org/wiki/Key:abandoned:

If you disagree with the consensus, we should discuss it at the OSM wiki, not here.

@westnordost
Copy link
Member

I used the lifecycle prefix because the simple tag is discouraged https://wiki.openstreetmap.org/wiki/Key:abandoned:

If you disagree with the consensus, we should discuss it at the OSM wiki, not here.

Right, I didn't notice it being discouraged as it is only mentioned in the continuous text.

@matkoniecz
Copy link
Member

matkoniecz commented Oct 27, 2020

Though at the same time simple tag is mentioned as valid tagging solution for some cases including building and landuse=quarry and say man_made=adit (though note that I was person who added this part in at least some pages, so do not count it as an independent support for my tagging ideas)

@peternewman
Copy link
Collaborator

At least in the UK, a lot of historic ruins will have an information board too, so you can see it's ruins and it used to be a manor house or a church or a watermill or whatever, even if it's just some wall outlines. Most other buildings are either still standing or have been knocked down and turned into houses etc.

@RiffLord
Copy link
Contributor Author

I've been reading all of the comments and thinking about the issue, while also using StreetComplete daily. I do agree that it makes sense to tag buildings based on what they are whenever possible, but there are cases when none of the available options feel appropriate (which is why I raised the issue in the first place).

At least in the UK, a lot of historic ruins will have an information board too, so you can see it's ruins and it used to be a manor house or a church or a watermill or whatever, even if it's just some wall outlines. Most other buildings are either still standing or have been knocked down and turned into houses etc.

This is obviously very convenient for users in the UK as it could allow them to correctly tag these buildings. However, it's not the case in many other areas. The area where I live is full abandoned buildings, historic or otherwise, many of which can't be identified based solely on the appearance.

In order to tag them I could:

  • Make an assumption based on the building's appearance. This approach increases the risk of error.
  • Ask around to see if anybody living nearby remembers what a particular building was. This isn't necessarily a bad idea, but a SC user should have all the options available to tag a building themselves within the app.

Let me show a couple of examples:

1Untitled

I happen to know that the above building used to be a restaurant, and as such I've tagged it so on StreetComplete. But just by looking at it there really is no way of knowing what it could have been.


2Untitled

The above building looks to me as if it could have been a house, but I have no idea for sure. It could have just as easily been a small office building, or something else entirely. StreetComplete asked me what kind of building this was, and I'm not sure how to tag it.


Both of these buildings have been abandoned for well over 15 years, and there's no sign that they will be demolished or restructured anytime soon. They are a defining part of the landscape just as much as the homes, stores and other buildings are.

To sum up, I agree that a building should be tagged based on what it is (or was), but my proposal is to add these abandoned or ruined categories for cases such as these, when the building's current state seems to be a defining factor more than what it originally was.

@matkoniecz
Copy link
Member

I happen to know that the above building used to be a restaurant, and as such I've tagged it so on StreetComplete. But just by looking at it there really is no way of knowing what it could have been.

By large windows/doors it seems clear to me that it is retail building (shop/restaurant) or maybe garage, how entrance looks like excludes garage (applies in my region, may not apply elsewhere)

The above building looks to me as if it could have been a house, but I have no idea for sure. It could have just as easily been a small office building, or something else entirely. StreetComplete asked me what kind of building this was, and I'm not sure how to tag it.

That is a great example.

@rhhsm
Copy link

rhhsm commented Oct 28, 2020

I think the best answer to give for these buildings is "Don't know". If you, as a local, can't figure out what kind of building it is, any user of your map data can't either. Maybe we should stop this discussion and go mapping :)

@westnordost
Copy link
Member

@rhhsm but what can be figured out is that it is abandoned and that is a data point that is worth recording.

@RiffLord
Copy link
Contributor Author

I believe that it would make sense to add these options to the quest, but to consider them special cases: while most of the options modify the building type tag, these should be implemented so as to add an abandoned=yes, ruin=yes or historic=yes tag, as suggested by @westnordost

A separate quest would be annoyingly redundant, while simply answering Don't Know is limiting, as it doesn't allow the user to insert some data which has value and which can be inserted into the OSM ecosystem.

@westnordost
Copy link
Member

So, you think these should be options in the "Other answers..." menu?

@RiffLord
Copy link
Contributor Author

I would put them in the Other category, the one with shed, bunker, service building, etc.

@westnordost
Copy link
Member

That's fine by me

@smichel17
Copy link
Member

I can imagine a future quest that asks, for abandoned buildings or ruins, if it is possible to determine the building's original purpose. I think that would address both sides here?

@peternewman
Copy link
Collaborator

This is obviously very convenient for users in the UK as it could allow them to correctly tag these buildings. However, it's not the case in many other areas. The area where I live is full abandoned buildings, historic or otherwise, many of which can't be identified based solely on the appearance.

Perhaps I was overly optimistic, or primarily concentrating on historic ruins @RiffLord . I agree I'd certainly struggle with the specifics of your second example.

@rhhsm
Copy link

rhhsm commented Oct 29, 2020

I used the lifecycle prefix because the simple tag is discouraged https://wiki.openstreetmap.org/wiki/Key:abandoned:

If you disagree with the consensus, we should discuss it at the OSM wiki, not here.

Right, I didn't notice it being discouraged as it is only mentioned in the continuous text.

You may want to wait on deciding on this until this openstreetmap/iD#8051 is settled.

@matkoniecz
Copy link
Member

matkoniecz commented Oct 29, 2020

You may want to wait on deciding on this until this openstreetmap/iD#8051 is settled.

This is an internal iD rendering issue with no relevance for tagging and not a place for OSM tagging discussions. Also, it is settled - closed and fixed.

@RiffLord
Copy link
Contributor Author

Forgive me for cluttering everyone's mailboxes, but I'm unsure where to ask this: I've implemented the changes on a separate branch and will be ready to make a pull request just as soon as I translate the string resources to the languages which I'm fluent in.

Should I merge the changes to the master branch before the pull request or will the repo owner do this afterwards?

@westnordost
Copy link
Member

just as soon as I translate the string resources to the languages which I'm fluent in.

You don't need and should not do that. Translations happen on POEditor, anything done in the repo will be overwritten by hte translations pulled from htere.

Should I merge the changes to the master branch before the pull request or will the repo owner do this afterwards?

Not sure what you mean. The PR should be mergeable, i.e. there should not be any conflicts with the current master.

@rhhsm
Copy link

rhhsm commented Oct 29, 2020

You may want to wait on deciding on this until this openstreetmap/iD#8051 is settled.

This is an internal iD rendering issue with no relevance for tagging and not a place for OSM tagging discussions. Also, it is settled - closed and fixed.

Thanks for your comments there and your edits on OSM wiki. For me it is important that SC follows consensus, not so much what that consensus is. Here it seems to depend mostly on software coding issues that I'm not qualified to comment on.

@RiffLord
Copy link
Contributor Author

You don't need and should not do that. Translations happen on POEditor, anything done in the repo will be overwritten by hte translations pulled from htere.

Alright, I won't do anything about that then.

Not sure what you mean. The PR should be mergeable, i.e. there should not be any conflicts with the current master.

I'm still quite new to the foss way of doing things, I just wanted to know who should perform the merge in this case.

@westnordost
Copy link
Member

Yes, you do the changes on a branch in a fork of this repository. Then, you create a new pull request (also known as merge request) to request merging your changes into this repository. This pull request is managed like an issue ticket, I can review it, comment on changes, request changes etc. and also finally merge it.

@peternewman
Copy link
Collaborator

I'm still quite new to the foss way of doing things, I just wanted to know who should perform the merge in this case.

Only people with maintainer rights (i.e. probably not you) will actually be allowed to merge it into this repo @RiffLord .

You can experiment/compare this on your own repo, if you or I open a PR into your version of it from one branch to another, you will see a merge button you can click to action the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants