Re: [PATCH v1] Re: [mkgmap-dev] maxspeed tag vs road_speed preset

2009-05-21 Thread Thilo Hannemann

Hi Mark,

It occurs to me that some people will want the maxspeed tag to  
override

the speed code in the rules file (obviously, the person who wrote that
code thought it was a good idea) but others, like Marco, would prefer
not to have that feature. So, I guess, we need (yet another) option to
control this behaviour. I attach a patch that does, essentially, the
same thing as your patch with the addition that you can switch that
behaviour on and off.


Thank you for that patch.

The patch I posted was intended to be applied locally if one wanted to  
remove that behaviour. With the state of mkgmap right now I think it  
is reasonable to hardcode such things as the maxspeed, because  
otherwise the style files will get overly complex and with the current  
syntax of the style files a lot of things can not be encoded at all  
(for example motorway exits, roundabouts etc.) For special map- 
tuning one will have to modify the mkgmap source. Which is probably  
even easier than coming up with highly complex style files to  
accomplish the same thing. So if you like to integrate the switch for  
the maxspeed handling into the trunk I will be happy, but I also  
understand that you might not want mkgmap to be cluttered with lots of  
special options.


Regards
Thilo

___
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev


[PATCH v1] Re: [mkgmap-dev] maxspeed tag vs road_speed preset

2009-05-21 Thread Marco Certelli
Hi, and thaks for your answer.

I'm not a developer so I really have some problem in applying the patch 
locally...

I just would like to keep control over the map generation. For example: in 
Italy the maxspeed out of the city is 90km/h. It applies to unclassified roads 
up to trunk roads (well, some trunks has 110 km/h).

I would like to code in the map in a way that a tertiary with 90km/h speed 
limit gets road_speed 3 (i.e 60 km effective speed), while a trunk with the 
same 90km/h limit gets road_speed 5 (i.e 90 km/h effective speed). You can 
immagine for the secondary and primary.

I looked at the source and you coded: if maxspeed40 then road_speed=3

It means that in the Italian cities (limit=50) the average speed coded in the 
map is 60km/h. It looks strange to me and carries to very optimistic city ETA 
calculations (and maybe not optimal route selection).

Moreover, I sow other map makers (DE:All_in_one_Garmin_Map) making maps 
including the maxspeed tag management in their styles (but maybe they use a 
patched mkgmap).

So, if it counts I do vote for the switch --ignore-maxspeed and I vote to 
integrate the
patch in the mainstream mkgmap (again, I'm not able to compile the code).

As you may have understood, I'm using Windo.%$/$£/$=)(/$))%$()%

Sorry, my PC crashed while I was writing 
Windo.=)/(/£%(//($/=)30/()/89

Ciao, Marco.




  ___
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Re: [PATCH v1] Re: [mkgmap-dev] maxspeed tag vs road_speed preset

2009-05-21 Thread Mark Burton

Hi Thilo,

 So if you like to integrate the switch for  
 the maxspeed handling into the trunk I will be happy, but I also  
 understand that you might not want mkgmap to be cluttered with lots of  
 special options.

Well, I don't have a problem with lots of options so I am happy to
commit that patch if people are agreed that it's useful.

Cheers,

Mark
___
mkgmap-dev mailing list
mkgmap-dev@lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev