Skip to content

fix a number of bugs and differences from openmaptiles vector tile schema - #825

Merged
systemed merged 1 commit into
systemed:masterfrom
cyclemap:master
Aug 29, 2026
Merged

fix a number of bugs and differences from openmaptiles vector tile schema#825
systemed merged 1 commit into
systemed:masterfrom
cyclemap:master

Conversation

@arichnad

@arichnad arichnad commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

fix a number of bugs and differences from openmaptiles vector tile schema

  1. zoom 4, 5, and 7 road values were incorrectly being used.

  2. openmaptiles shows admin 3 boundaries much higher.

  3. unpaved and paved values are parsed from surface list

  4. remove duplicate variable definitions.

  5. SetMinZoomByAreaWithLimit allows up to minzoom 4

  6. fix whitespace and a few variable issues.

  7. SetMinZoomByAreaWithLimit improvements (allow zoom level 1 and above)

  8. recalculate a few constants to use 40.075e6 meters

  9. do not set an unset name.

  10. separating out minor road values and path road values allows for a more correct interpretation of the zoom levels.

Comment thread resources/process-openmaptiles.lua
@arichnad

Copy link
Copy Markdown
Contributor Author

I have fixed the merge conflict.

@arichnad

Copy link
Copy Markdown
Contributor Author

I have fixed the merge conflicts. And updated the description above. Let me know what you think, thanks!

…hema

1.  zoom 4, 5, and 7 road values were incorrectly being used.

2.  openmaptiles shows admin 3 boundaries much higher.

3.  unpaved and paved values are parsed from surface list

4.  remove duplicate variable definitions.

5.  SetMinZoomByAreaWithLimit allows up to minzoom 4

6.  fix whitespace and a few variable issues.

7.  SetMinZoomByAreaWithLimit improvements (allow zoom level 1 and above)

8.  recalculate a few constants to use 40.075e6 meters

9.  do not set an unset name.

10.  separating out minor road values and path road values allows for a more correct interpretation of the zoom levels.
@arichnad

Copy link
Copy Markdown
Contributor Author

I have fixed the merge conflicts, thanks!

@systemed
systemed merged commit 9de20be into systemed:master Aug 29, 2026
11 of 12 checks passed
@systemed

Copy link
Copy Markdown
Owner

Thank you and sorry for the slow merge!

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