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

Street crossing island quests only showing during scan (and on edges of scan area) #2430

Closed
dbdean opened this issue Dec 28, 2020 · 7 comments
Labels

Comments

@dbdean
Copy link
Contributor

dbdean commented Dec 28, 2020

When StreetComplete is scanning for quests, crossing island quests show up in my area, but they mostly disappear before the scanning is over. Sometimes they stay on the boundaries of what appears to be the scanning area though. These quests will disappear if I scan an area closer to them, so they are difficult to actually visit and answer.

From discussions when the island quest was implemented, it may be intentional that they shouldn't show because I have most of the footpaths in my area mapped separately to the roads. However I think they should be showing, as many of these crossing in town do currently have islands and I'd like to survey them.

(Perhaps a separate issue) Can we change the above logic to always ask for crossing islands on roads that aren't one-way, or have more than 1 marked lane, perhaps?

Example, crossings just outside scan area:

Screenshot_20201229-082252

And after a new scan is run centered on this area:

Screenshot_20201229-082339

For reference area is Armidale, NSW, Australia.

@peternewman
Copy link
Collaborator

From what I can see your middle left one in the top image is this node @dbdean :
https://www.openstreetmap.org/node/5391039082

I'd agree I get the same behaviour on some adjacent ones.

It looks like it's getting excluded because this way:
https://www.openstreetmap.org/way/558914148

Matches the exclusion:

private val excludedWaysFilter by lazy { """
ways with
highway and access ~ private|no
or highway and oneway and oneway != no
or highway ~ path|footway|cycleway|pedestrian
""".toElementFilterExpression()}

See #2030 for some of the reasoning behind it, but I'd agree it does look rather aggressive at filtering out nodes currently.

As to why the others disappear, I'd imagine the way was outside the search box when it's at the periphery, hence it's shown, until you re-center then the way is also found and it gets dropped.

@westnordost
Copy link
Member

I also looked at the nodes you mentioned. The quest should not be shown for the mentioned nodes because it is a vertex of a highway=footway. Why is it still shown during the download process?

Right now, I have no idea!

@peternewman suggested that the footway might not have been part of the download bounding box. But to my understanding, this is impossible, as the OSM API should return all ways who got at least one vertex in the bounding box. So if the vertex is in the bounding box, the way should also be in the bounding box.

image

@westnordost
Copy link
Member

Ah of course, thats' how:

image

image

@westnordost
Copy link
Member

Ok I'd say the solution is to not create quests for elements whose center is outside of the download bounding box.

@peternewman
Copy link
Collaborator

Heh, so I was 180 degrees wrong then! 😄

I had a look through the PR when it was created, but looking at @dbdean 's screenshots IMHO its being rather aggressive at filtering to try and avoid spam in some places, those all clearly look like they have the potential to have islands and aren't part of a crossing mapped in detail.

@dbdean
Copy link
Contributor Author

dbdean commented Dec 30, 2020

Thanks for the detective work. Now these quests won't be teasing me anymore. However, I think these quests are reasonable in this circumstance, so I think I'll create a new issue to see if we can work out how they can be included.

@dbdean
Copy link
Contributor Author

dbdean commented Jan 3, 2021

New quest requesting change in logic to allow for crossing island quests on roads with connected footways is up at #2454.

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

No branches or pull requests

3 participants