Skip to content

Landuse geometry simplification#500

Open
wipfli wants to merge 7 commits into
mainfrom
landuse3
Open

Landuse geometry simplification#500
wipfli wants to merge 7 commits into
mainfrom
landuse3

Conversation

@wipfli

@wipfli wipfli commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator

Is an alternative way to reduce tile size in the landuse layer. Instead of dropping kind values, we run some geometry post processing.

Uses Visvalingam Whyatt simplification.

When running on Switzerland, the landuse-only archive size goes from 171 MB (100%) on main to 110 MB (64%) on this branch. Projected space savings on planet 35 GB * (1 - 0.64) = 12.6 GB.

@wipfli

wipfli commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator Author

Ugh this is #500. Congrats @bdon!

@wipfli

wipfli commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator Author

@wipfli

wipfli commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator Author

Somehow geometries are broken at zoom levels maybe 10 to 14

image
https://pub-cf7f11e26ace447db8f7215b61ac0eae.r2.dev/protomaps-500/index.html#map=12.12/47.65143/8.06254

@bdon

bdon commented Jun 5, 2025

Copy link
Copy Markdown
Member

Odd... first thought is maybe the snapAndFixPolygon is not being called at the end of the VW + smoothing simplification?

@wipfli

wipfli commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator Author

image
need to update this render test...

@wipfli wipfli changed the title Landuse geometry improvements to reduce tile size Landuse geometry simplification Jun 5, 2025
@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2025

Copy link
Copy Markdown

@wipfli

wipfli commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator Author

I have updated the demo at https://pub-cf7f11e26ace447db8f7215b61ac0eae.r2.dev/protomaps-500/index.html#map=9.76/47.2065/8.2466 with just some data for Switzerland.

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.

2 participants