Skip to content

Promote / demote US roads #423

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Promote / demote US roads #423

wants to merge 2 commits into from

Conversation

wipfli
Copy link
Collaborator

@wipfli wipfli commented Mar 14, 2025

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
Copy link
Member

bdon commented Mar 17, 2025

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

@nvkelso
Copy link
Collaborator

nvkelso commented Mar 17, 2025

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
Copy link
Member

bdon commented Mar 17, 2025

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

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

@nvkelso
Copy link
Collaborator

nvkelso commented Mar 17, 2025

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
Copy link
Collaborator Author

wipfli commented Mar 17, 2025

@wipfli wipfli mentioned this pull request Mar 17, 2025
@wipfli wipfli marked this pull request as ready for review March 17, 2025 13:04
@wipfli
Copy link
Collaborator Author

wipfli commented Mar 17, 2025

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
Copy link
Collaborator

nvkelso commented Mar 18, 2025

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

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

Successfully merging this pull request may close these issues.

3 participants