[mkgmap-dev] [patch v1] link-pois-to-ways and restrictions

Fri Dec 27 14:22:16 GMT 2013

Hi WanMil,

> > I think the wanted behaviour is this:
> > 1) The change in speed should be applied to max. 25m
> > before and after the POI or to the connection
> > with another way, whatever comes first.
> > 2) A POI like a barrier should not change access
> > to the way before or after the POI, only at the POI.
> >
> > The attached patch implements this changed behaviour.
> > The type 2)  POI are converted to route restrictions.
> 
> I've tested part 2) of the patch and it worked fine for me!

well, I think that means that part 1) is not yet working. 
I am not sure, but I think you moved the evaluation of the
tags mkgmap:road-speed and mkgmap:road-class 
so that the interpretation of the POI has no effect.
Was that intended?

> 
> 
> >
> > The patch also changes the meaning of route restrictions.
> > In r2907 and before, route restrictions prohibit access for
> > pedestrians and emergency, the patched version
> > allows them by default.
> 
> Does that mean a common turn restriction from a=>b also means that 
> pedestrians never can route a=>b?

yes, in trunk all turn restrictions are created without an exception
for pedestrians or  emergency.  

> >
> > If I hear no complains, I'll commit that next sunday.
> 
> I propose to move the link-pois-to-ways handling before merging roads. 
> Do you think it's complicated to move it?

Ahh, I forgot about this. I got a cold last weekend, so I'm not able to concentrate
very well. If I got this right, it should work if we add new 
instances to roads  and roadTypes for each part of the way, maybe with 
a fake id. Not sure if it will reduce the number of roads,
I think more important is to fix the above problem.

Gerd

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20131227/ec2521d1/attachment.html>