Ticket #191 (closed defect: fixed)

Opened 20 months ago

Last modified 20 months ago

PolygonPatternSymbolizer broken with Cairo

Reported by: jburgess777 Owned by: tomh
Priority: normal Milestone: 0.6.0
Component: Core Library Version: 0.5.1
Severity: Normal Keywords:
Cc: Patch Needs Improvement: no
Needs Docmentation: no Has Patch?: no
Design Decision Needed: no

Description

I had a report that the landuse=forest are not being shown in the PDF output of the OSM export tab. I tried some local rendering and it seems that even PNGs rendered using Cairo are missing any rendering of forests. The same area rendered with Agg is fine.

e.g. this forest on the IOW http://www.openstreetmap.org/?lat=50.66261&lon=-1.38956&zoom=15&layers=B000FFF

It seems that perhaps the PolygonPatternSymbolizer is not working correctly with Cairo.

Attachments

mapnik-fix-cairo-polygon-patterns.patch (382 bytes) - added by jburgess777 20 months ago.
This patch seems to fix the forest rendering

Change History

Changed 20 months ago by tomh

Well it certainly was working when I wrote it, so if isn't working now then somebody broke it.

Changed 20 months ago by tomh

Cemetries are working, so it must be something more complex than just all patterned polygons failing.

Changed 20 months ago by jburgess777

I notice forest.png is only RGB, whereas the graveyard is RGBA I'll try adding some alpha to the forest.png

Changed 20 months ago by jburgess777

This patch seems to fix the forest rendering

Changed 20 months ago by artem

  • status changed from new to closed
  • resolution set to fixed

Applied in r821.

Changed 20 months ago by tomh

Um... Why exactly does that patch work? Specifically, why were graveyards already working?

Changed 20 months ago by jburgess777

The graveyard / cemetery here does not appear in the PDF output: http://www.openstreetmap.org/?lat=55.88669&lon=-5.26222&zoom=16&layers=B000FTFT

We seem to have overlapping zoom ranges for the plain and patterned grave_yard styles. Perhaps you are seeing the plain polygon symbolizer?

Note: See TracTickets for help on using tickets.