Skip to content

Promote / demote US roads#423

Open
wipfli wants to merge 2 commits into
mainfrom
promote-demote-us-roads
Open

Promote / demote US roads#423
wipfli wants to merge 2 commits into
mainfrom
promote-demote-us-roads

Conversation

@wipfli

@wipfli wipfli commented Mar 14, 2025

Copy link
Copy Markdown
Collaborator

Alternative solution to #422, fixes display fragmentation of roads around zoom 6 in the US.

Has not been tested yet. I will upload an archive later.

@bdon

bdon commented Mar 17, 2025

Copy link
Copy Markdown
Member

I think this solution is better than #422 since there are less vertices/features in the tiles.

@nvkelso

nvkelso commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator

I'm curious to see before / after / comparisons between the approaches :)

Alternative solution to #422, fixes display fragmentation of roads around zoom 6 in the US.

Is this because the style in the other PR needs a paired update?

I think this solution is better than #422 since there are less vertices/features in the tiles.

What's removing the vertices / features? Feature merging elsewhere?

@bdon

bdon commented Mar 17, 2025

Copy link
Copy Markdown
Member

Before:
CleanShot 2025-03-17 at 11 57 28@2x

After:
CleanShot 2025-03-17 at 11 57 38@2x

@nvkelso

nvkelso commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator

This smells like a styling issue where the major_roads are too faint compared with the highways.

Overzooming the images shows the same geometry vertexes are included in each before and after. The after draws them with a thinner styling.

Going with this this data hack approach risks either creating flicker on the next zooms in when the real raw classification is shown, or perpetuates the reclassification giving false impression of the road classes (sometimes ok if just for early zooms). Spice to taste.

@wipfli

wipfli commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator Author

@wipfli wipfli mentioned this pull request Mar 17, 2025
@sonarqubecloud

Copy link
Copy Markdown

@wipfli wipfli marked this pull request as ready for review March 17, 2025 13:04
@wipfli

wipfli commented Mar 17, 2025

Copy link
Copy Markdown
Collaborator Author

I think styling the roads based on the kind attribute is the right thing to do. It represents the kind of road. Important ones show up earlier.
The highway tag in OpenStreetMap on the other hand represents more the class of road in the national network and not necessarily the importance of the road. For example in the US, if a highway=primary is part of the US:US network, then that road is more important than a highway=primary which is not part of any network. So the highway tag alone does not contain the full road importance...

@nvkelso

nvkelso commented Mar 18, 2025

Copy link
Copy Markdown
Collaborator

No doubt it looks better at zoom 7! But by zoom 9 these roads should be showing their true colors. With this PR it looses detail at that zoom (and higher).

Before:
image

After:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants