streetcomplete / StreetComplete

Easy to use OpenStreetMap editor for Android
https://streetcomplete.app
GNU General Public License v3.0
3.89k stars 355 forks source link

Building-Type Quest: This ist a building:part #3351

Closed DeusFigendi closed 3 years ago

DeusFigendi commented 3 years ago

In the area where I live there are many building parts that a mapped as own buildings. And of coarse StreetComplete asks for every "building" what type (and number of levels and roof shape) it is/has.

I would love to have an answer like "this is a building:part not an own building" as "other answers".

For example this building is in fact a building part I think, as one might see in this photography

The best outcome for me would be if I could choose "this is a building:part" and streetComplete asks me of what building this is a part of. And then extends the chosen building to the form of the building I was asked for and makes that one to a bulding:part

Okay that was complicated, I'll draw something… Rastergrafik

Well, maybe this is too complicated (the extension of the building) in that case it would also be fine to change the building to a building:part and leave kinda fixme that says "this is part of another building (with ID 123456) please join those". And maybe this is a very special problem that only occurs in my region but in occurs massively here!

mnalis commented 3 years ago

It does indeed look too complicated for StreetComplete, IMHO. There is an option Other answers... / Can't say, where you can then leave a note (with a picture, if needed) explaining the problem, so it can be fixed later with more general editor.

Or you might consider also installing some more general-purpose mobile editor (like Vespucci) for more advanced editing tasks like that, if you prefer doing it yourself on the ground. It can be interfaced with StreetComplete too - just long press on the problematic area of the screen, and choose Open location in another app

matkoniecz commented 3 years ago

And maybe this is a very special problem that only occurs in my region but in occurs massively here!

I have seen it few times in various places but is not very common (less than 50 occurrences in total?). So your area may be hit especially hard, while implementing this feature may be not worth doing this :(

Unlike way splitting that was also massive effort, but is useful and in significant use everywhere.

DeusFigendi commented 3 years ago

It does indeed look too complicated for StreetComplete, IMHO. There is an option Other answers... / Can't say, where you can then leave a note (with a picture, if needed) explaining the problem, so it can be fixed later with more general editor.

Or you might consider also installing some more general-purpose mobile editor (like Vespucci) for more advanced editing tasks like that, if you prefer doing it yourself on the ground. It can be interfaced with StreetComplete too - just long press on the problematic area of the screen, and choose Open location in another app

Indeed I do both XD But if there are dozens and dozens of those cases in one street it's mh well a little odd to do a note on many of those, but just putting one note also doesn't fit well, because at home (maybe a few days later…) I couldn't remember which ways are building parts and which ain't. And other mapper also can't know.

About Vespucci… yes I use it but when I use StreetComplete than often because I don't have the time to do complex mapping work with vespucci ^^

So your area may be hit especially hard, while implementing this feature may be not worth doing this :(

Yea, might be. That's why I suggested just to change the building=yes tag to a building:part=samevalue and leave the correction of the erm… "mothers building" to other mappers later.

westnordost commented 3 years ago

The UI could work and it would be possible to implement. However, it is quite some work. Especially the edge cases like

image

will complicate this a lot. So, I will not do this, too bad ratio between gain and effort.

In case you want to do this or someone else reading this, the structure can be similar to split ways. See

RubenKelevra commented 2 years ago

@westnordost I understand that a "full fix" is impossible. But does this mean that there should not be a partial fix?

I mean if there are two simple rectangulars with two common nodes SC could easily merge them into a building and make the building:part thing itself.

If that's not possible for SC it could create an note, with both buildings mentioned and ask the user to provide further evidence etc.

Currently there's no such thing in the whole UI - so the users of SC don't know nothing about building parts. This is by far the worst "solution", as the users will just create more info on the "secondary building" and even try to find the house number and mark that it doesn't have one etc.

matkoniecz commented 2 years ago

I mean if there are two simple rectangulars with two common nodes SC could easily merge them into a building and make the building:part thing itself.

If that's not possible for SC it could create an note, with both buildings mentioned and ask the user to provide further evidence etc.

Both will result in confused users why it works only sometimes, and would require very significant work to get something broken by design.

scy commented 10 hours ago

I am by no means a beginner OSM user, but StreetComplete's behavior in this regard has led me to completely disable the "building type" quest.

While many users in this thread seem to have not come across building:part often, it's a very common thing where I live (rural Niedersachsen in Germany), and I suspect this is because the building shapes have been imported from government open data that way, i.e. separate ways for what are essentially parts of the same house.

I understand that being able to split buildings, or merge them, might be out of scope for StreetComplete. But the way it is right now is partially wrong, and at least confusing and irritating, even for experienced users.

My suggestion consists of two parts:

  1. Do not request building type if the way is tagged as building:part=yes.
  2. Allow giving "this is part of an adjacent, already tagged building" as an answer to the building type quest, which will then set building:part=yes on the way.

What do you think?

matkoniecz commented 9 hours ago

and I suspect this is because the building shapes have been imported from government open data that way

if you are in area with badly done import: I would open notes for such bogus data

Even if I tag them with building:part=yes

are you removing invalid building=yes tag?

matkoniecz commented 9 hours ago

Allow giving "this is part of an adjacent, already tagged building" as an answer to the building type quest, which will then set building:part=yes on the way.

this is typically an insufficient fix - opening notes is better (and yes, it can be painful if you are in area with badly done import, some city in Poland had import that added each step in external staircases as separate tiny building...)

scy commented 8 hours ago

are you removing invalid building=yes tag?

Oh! I am not. (Well, to be exact, they have been tagged by other people.) The main part of the building is tagged building=<whatever>, the parts attached to it with building:yes and building:part=yes, but there's no way around all of the parts that's tagged with building=<whatever>.

Like, each way is tagged with building=*, while the non-main parts additionally have building:part=yes.

I see now what the issue is and will try re-tagging these as I come across them, thanks for the hint!

matkoniecz commented 8 hours ago

but there's no way around all of the parts that's tagged with building=.

Ideally this one also be fixed and main building part also tagged as one more building part

Yes, if it is badly done import it will take a lot of work - but maybe it is possible to automate this? It may be worth asking on forums

Good news is that it affects not only SC, so fix will help not only here