Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

landuse=military on meta-tile boundaries #97

Closed
pnorman opened this issue Aug 6, 2013 · 4 comments · Fixed by #196
Closed

landuse=military on meta-tile boundaries #97

pnorman opened this issue Aug 6, 2013 · 4 comments · Fixed by #196

Comments

@pnorman
Copy link
Collaborator

pnorman commented Aug 6, 2013

At the intersection of four meta-tile boundaries there is a mismatch in hatching for z15 and below

15/7575/1250315/7576/12503
15/7575/1250415/7576/12504

Similar results on z16, z17, z18, but z14 is fine.

I have a theory as to why this is occurring, and how to fix it.

A z15 meta on orm is a z12 tile + 16 z12 pixel buffer. The buffer doesn't matter for what I think is wrong.

The areas covered by the metas are:
12/946/156212/947/1562
12/946/156312/947/1563

I believe polygon-clip: false may fix this. The description says

geometries are clipped to map bounds by default for best rendering performance. In some cases users may wish to disable this to avoid rendering artifacts."

cc @ToeBee

@gravitystorm
Copy link
Owner

Great investigation.

polygon-pattern-alignment: global is the preferred fix, along with ensuring the pattern is 256px repeating. If we can keep the clipping on, we get a (small) performance boost compared with switching it off.

@pnorman
Copy link
Collaborator Author

pnorman commented Aug 6, 2013

Does it need to be 256px repeating or just fit an integer number of times into 256px? (e.g. 128px, 64px, etc?)

@gravitystorm
Copy link
Owner

Integer number of times is fine.

@springmeyer
Copy link
Contributor

yes polygon-pattern-alignment: global sounds like the right fix.

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

Successfully merging a pull request may close this issue.

3 participants