Quarry pattern is not periodic any more

[imagico]

http://www.openstreetmap.org/#map=19/48.07891/7.70540

due to #2683, likely the same problem as in #2685.

[kocio-pl]

Thanks for noticing! I think we can probably safely revert this PR. Fixing problems with Mapnik rendering of SVG would be much better, of course.

[kocio-pl]

I've just found this (#937 (comment)):

Without a fixed metatile size, this implies both pattern dimensions should be a multiple of two, and at most 256.

This SVG is 512 px. Could it be a problem here? Why 256 should be max size for patterns?

[sommerluk]

I suppose this can be easily solved by re-creating the pattern with the most recent version of jsdotpattern, that (since last sunday) also adds a dummy geometry when sanitizing, right?

[kocio-pl]

I was not aware of the change, but it works. I can merge fix, but there's still a question about all 512 px patterns.

[sommerluk]

Anyway: As the current pattern is a regular pattern I suppose that’s not a big deal. Or course, it will round the distance to something that’s a divisor of 256, so the symbal distance might be slightly bigger or smaller, but likely < 10% difference.