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

Mysterious text below some settings in v8.0 #1215

Closed
cyclingcat opened this issue Sep 26, 2018 · 17 comments
Closed

Mysterious text below some settings in v8.0 #1215

cyclingcat opened this issue Sep 26, 2018 · 17 comments
Labels

Comments

@cyclingcat
Copy link

Since v8.0, some settings concerning quest selection display an additional text in a grey box below the setting's text: "Niemals in Europäische Union gezeigt" ("Never shown in European Union" or similar, see screenshot). This puzzles me, as I am located in Germany and currently see a lot of those quests which show this grey box. What's going wrong here?

sc_settings

For testing purposes, I just reverted to v7.1 on one of my devices and these grey boxes never appeared since. (I have also never noticed it before.) Updated to v8.0 again and the grey boxes reappeared immediately.

The cycling cat

@ENT8R
Copy link
Contributor

ENT8R commented Sep 26, 2018

Related to #1150.

@westnordost
Copy link
Member

westnordost commented Sep 26, 2018

Where did you focus your screen on when opening the settings?
(Best, exact location)

@westnordost westnordost added bug feedback required more info is needed, issue will be likely closed if it is not provided labels Sep 26, 2018
@cyclingcat
Copy link
Author

I don't want to disclose my home here publicly, but it's in the south of Germany, more than 100 km away from any border. The screen shows just my home and approx. 100 meters around. Let me know if you need the exact location, I will send it to you via e-mail.

@westnordost
Copy link
Member

It would be better. But you can also scroll somewhere else and try there again.

@westnordost
Copy link
Member

Also, are cycleway quests (etc) then not shown in your area?

@cyclingcat
Copy link
Author

The cycleway quests are shown despite the grey box in the settings.

@cyclingcat
Copy link
Author

Now it's getting even more mysterious: I scrolled around, just some kilometres away from my GPS location, looked into the settings, and the grey boxes are gone! Went back to the GPS location (by tapping on the button in the lower right corner): The boxes reappeared. Scrolled around just some metres: Boxes are still there. Scrolled around a little further away: Boxes are gone.

I will try to find a pattern behind all this.

@westnordost
Copy link
Member

Perhaps you should send me the exact GPS position, it would be better. [email protected]

@cyclingcat
Copy link
Author

It looks really random. Even when scrolling through Switzerland (no parts of other countries shown on the map), the box sometimes appears, containing the string "European Union" (!) or "ATLANTIS" (!!)

@matkoniecz
Copy link
Member

@cyclingcat Please send/post one exact position where it happens. This way @westnordost will be able to check what happens.

BTW, Atlantis is easter egg that should appear when location is on ocean.

@westnordost
Copy link
Member

Can reproduce. This is a bug in the countryboundaries library.... :-(

Introduced in this commit: westnordost/countryboundaries@87eb02f

Apparently the winding number algorithm has some bug that (noone) knows about. Need to set a breakpoint and go through this step by step.

@westnordost westnordost removed the feedback required more info is needed, issue will be likely closed if it is not provided label Sep 26, 2018
@westnordost
Copy link
Member

The winding number algorithm is not from myself, I copied the code from its source. Apparently it is either inherently flawed in some way or another, or the polygons I am working with are somehow invalid (enough) for that algorithm to get confused.
If someone else wants to look into that, you are very welcome. The algorithm is as much as a black box to me than to anyone else. (Well, I understand its intention but apparently there is an edge case that is not accounted for)

@rugk
Copy link
Contributor

rugk commented Sep 26, 2018

So time for a "help wanted" tag?

@westnordost westnordost added help wanted help by contributors is appreciated; might be a good first contribution for first-timers and removed help wanted help by contributors is appreciated; might be a good first contribution for first-timers labels Sep 26, 2018
@westnordost
Copy link
Member

Found it. An integer overflow.

@ENT8R
Copy link
Contributor

ENT8R commented Oct 4, 2018

It seems like this issue caused some more problems: According to https://forum.openstreetmap.org/viewtopic.php?pid=718691#p718691 which mentions https://www.openstreetmap.org/changeset/63068614 there were some edits in Germany which set maxspeed:type=NL:urban in a city in Germany. And in the same changeset there were also some edits which added maxspeed:type=default:urban which makes absolutely no sense to me... Is this also caused by this issue? If yes, we should maybe block version 8.0 from uploading corrupted data?

@matkoniecz
Copy link
Member

@ENT8R I think that data corruption may deserve a new issue.

@westnordost
Copy link
Member

It's okay, I am subscribed to the whole project, inclusive all closed tickets. I added 8.0-beta1 and 8.0 to the block list, and will create a ticket to organize a repair/check of the wrong data now.

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

5 participants