Opened 10 years ago

Closed 10 years ago

#2230 closed defect (fixed)

Double parking "P"

Reported by: wiso Owned by: jburgess777@…
Priority: minor Milestone:
Component: mapnik Version:
Keywords: parking, P Cc: giurrero@…

Description

Look here:

http://www.openstreetmap.org/?lat=45.356127&lon=9.65366&zoom=18&layers=B000FTF

there is a parking with 2 "P". I try to regenerate the tiles locally with R17299 and I don't get the double "P"

Change History (5)

comment:1 Changed 10 years ago by Tom Hughes

Owner: changed from mapnik-team@… to jburgess777@…

Jon - do we still have the old hack to add P markers installed? If we do I guess it is clashing with the new code to render markers for areas...

comment:2 Changed 10 years ago by Ldp

Yes, that one is still active (or actually, in osm2pgsql's source). I was actually counting on it picking about the same spot as the polygon centroid mapnik is supposed to take, dropping one of them due to a collision.

I wonder which P comes from which source. If the out of center one is mapnik's, why would it choose such an awkward spot for such a simple polygon?

comment:3 Changed 10 years ago by jburgess777@…

Resolution: fixed
Status: newclosed

The two nodes on the right have been moved several times. If they were aligned with the road to right then the second P would be in about the centre of this area. I don't believe the diff update process does anything to delete the generated P nodes when parking areas are updated.

There are currently two generated P nodes in the DB:

gis=> select osm_id,amenity,astext(transform(way,4326)) from planet_osm_point where osm_id=26267629;
  osm_id  | amenity |                  astext
----------+---------+------------------------------------------
 26267629 | parking | POINT(9.65349861966763 45.3561133580288)
 26267629 | parking | POINT(9.65370460336228 45.3561374715661)
(2 rows)

I have now deleted them both. This only effects the Mapnik rendering DB, not the OSM ways or nodes.

comment:4 Changed 10 years ago by larza@…

Priority: majorminor
Resolution: fixed
Status: closedreopened

comment:5 Changed 10 years ago by Ldp

Resolution: fixed
Status: reopenedclosed

The last full planet import was Sept 2, so you saw an old tile when you reopened this ticket. Those areas now have 1 P symbol.

Note: See TracTickets for help on using tickets.