Helium314 / SCEE

OpenStreetMap surveyor app for experienced OSM contributors
GNU General Public License v3.0
142 stars 10 forks source link

Quest Trail Visibilitity on private path #570

Closed ravenfeld closed 2 months ago

ravenfeld commented 3 months ago

While carrying out quests for sac_scale and trail visibility I realised that I was being asked for information on private paths.

Maybe add this for everyone who plays this quest?

      and access !~ no|private
      and foot !~ no|private
mcliquid commented 3 months ago

Yes, that should be a good idea, as one should assume that if you are permitted or have private access, you won't be able to get there. Super experts can still remove the filter themselves in the quest settings.

mnalis commented 3 months ago

Generally the idea might seem OK @ravenfeld but I'm afraid such implementation is much too simple (unless I'm misreading it)

For example, there exist tag combinations like:

etc. which should all indicate it is OK for OSM mapper to legally be there and do the mapping

Yet such simple filter would seem to skip all of them?

While such oversimplification perhaps might be acceptable loss/gain ratio for vanilla StreetComplete, it doesn't feel as a good fit SCEE-specific quests, IMHO, as its users should be quite familiar with "don't enter private property" and other OSM mapping rules and often want to map all things they are allowed to, instead of too simple filter incorrectly hiding more than a third[^2] of such tag combinations.

Yes, in SCEE mappers can manually override quest selectors, but IMHO we still should tailor SCEE-specific Quests towards average SCEE user. Majority of which I think would definitely like to see paths tagged with access=no + foot=yes (and similar...)


Possible solutions:

[^1]: yeah, some folks do map from bicycles. I personally would recommend you always stop+map if you do that (unless you don't mind some self-inflicted damage occasionally :smiling_face_withtear:) . Some others map from busses, trains, as passengers in a car etc. too, but that is not applicable for those two specific quests (but might apply to other Quests) [^2]: I have not done detailed analyses of the tag usage, but just quickly guesstimated by looking at taginfo for access=no combinations, of which more than 35% "no"_ values were overridden by more specific access tags (like foot=yes, bicycle=designated etc).

ravenfeld commented 3 months ago

You're absolutely right, I got the code from AddCycleway and I noticed in my tests that my paths had no combination. So I'm going to fix that. Thank you very much

ravenfeld commented 3 months ago

@mnalis

I'll test your cases tomorrow, but it seems to me that's what you want?

( access !~ no|private or foot ~ yes|permissive|designated or bicycle ~ yes|permissive|designated)

I didn't understand snowmobile. Do you want me to take care of it? We're talking about hiking trails. Maybe let his people amend the application if that's the case no?

Helium314 commented 3 months ago

( access !~ no|private or foot ~ yes|permissive|designated or bicycle ~ yes|permissive|designated)

This looks like a reasonable (default) restriction to me.

mnalis commented 2 months ago
( access !~ no|private or foot ~ yes|permissive|designated or bicycle ~ yes|permissive|designated)

This looks like a reasonable (default) restriction to me.

It would work for me personally, but I guess that's because I'm always either on foot or on bicycle while mapping paths. But as you say, this is SCEE, so sub-optimal default is not a huge deal, as any mapper can change it to their own preferences.

We could go with that and extend it later if people complain...

I didn't understand snowmobile. Do you want me to take care of it? We're talking about hiking trails

Well, we are mostly talking about highway=path, which includes hiking trails, yes, but also includes many other things. One of which is mentioned snowmobile paths. I can't tell you much about them, as they don't really exist over here (Croatia is more well known for its sunny beaches, than its snow), but closer to the North pole snowmobiles are a thing (and I guess trail visibility on them might be as useful as it might be hiking trails and bicycle paths)