A road section disappears on lower zooms #10107

Open
opened 2025-01-21 09:45:52 +00:00 by pastk · 1 comment
Member

https://www.openstreetmap.org/way/172544089#map=15/48.10010/37.94158
Its highway=secondary so should be visible from z10-.

But the section disappears on z12 and earlier (Android device):
Image

What's weird in that on the linux desktop its rendered on z12, but is absent on z10-11 still (the maps are the same of course)
Image

So it could be a bug of the lines simplification algo in the generator (produces an invalid result) or a rendering bug in the drape engine.
@vng FYI

https://www.openstreetmap.org/way/172544089#map=15/48.10010/37.94158 Its `highway=secondary` so should be visible from z10-. But the section disappears on z12 and earlier (Android device): ![Image](https://github.com/user-attachments/assets/660e10b0-5ebe-46f9-9f85-9359e6615885) What's weird in that on the linux desktop its rendered on z12, but is absent on z10-11 still (the maps are the same of course) ![Image](https://github.com/user-attachments/assets/2ab00306-9dbc-4b77-801b-fa2a4b45d32a) So it could be a bug of the lines simplification algo in the generator (produces an invalid result) or a rendering bug in the drape engine. ` @vng` FYI
Author
Member

Looks like a bug of the extra run-time simplification (ref ApplyLineFeatureGeometry::operator()).

The way is almost circular (the start and the end are very close to each other). So at some point the algo decides to make a short-cut between them and skip all other points of the spline :)

The affected zoom level is different for various devices because they have different dpi, which affects the simplification thresholds.

IMO this run-time simplification is redundant and should be dropped.

Looks like a bug of the extra run-time simplification (ref `ApplyLineFeatureGeometry::operator()`). The way is almost circular (the start and the end are very close to each other). So at some point the algo decides to make a short-cut between them and skip all other points of the spline :) The affected zoom level is different for various devices because they have different dpi, which affects the simplification thresholds. IMO this run-time simplification is redundant and should be dropped. - related to https://git.omaps.dev/organicmaps/organicmaps/pulls/6122
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps#10107
No description provided.