
Hi Marko,
the patch (this v1) is only meant to solve routing problems caused by rounding I don't think that it has an influence on SeaGenerator.
Right, I was too optimistic, hoping that this would fix the bogus warnings in SeaGenerator too.
Maybe it is possible to use the higher precision for this as well. I assume we just have to use it when converting to java areas in Java2DConverter.
The patch (or some other change since r2795) did make 2 of 4 warnings for roundabout flare roads go away. Only the following remained:
2013/10/31 21:17:17 WARNING (StyledConverter): 63240002.osm.pbf: Roundabout segment 35062431 direction looks wrong (see http://www.openstreetmap.org/?mlat=60.23825&mlon=24.88777&zoom=17) 2013/10/31 21:17:18 WARNING (StyledConverter): 63240002.osm.pbf: Roundabout segment 34151581 direction looks wrong (see http://www.openstreetmap.org/?mlat=60.23242&mlon=24.95879&zoom=17) 2013/10/31 21:17:37 WARNING (RouteNode): 63240005.osm.pbf: Outgoing roundabout flare road (4;9;13;23 Vaajakoskentie, http://www.openstreetmap.org/browse/way/221616945) points in wrong direction? http://www.openstreetmap.org/?mlat=62.24637&mlon=25.87302&zoom=17
Both roundabout segments are in micro-mapped roundabouts that have been split because of route relations.
I reverted your patch and reran with today's dump. I got again the usual 5 roundabout warnings (2 more about flare roads) than the above. So, your patch is making a positive difference there.
OK, that's what I expected :-) As I said, the patch was a quick hack, I just wanted to demonstrate that we only have to change a few lines to store the higher precision, and it will not cost more memory.
Do you have a reason for not using precompiled sea?
Yes. I mainly use mkgmap as a validator, and I am routinely fixing errors in finland.osm.pbf. The sea polygons are not severely broken that often, it is usually just anti-islands. Only about once per month I have to fire up JOSM with an Osmosis-made extract of natural=coastline to see what is wrong.
OK, I see. I'll try with the same data to check what can be changed. Gerd