Hi all, I've just noticed that this patch was not yet applied. If nobody complains I'll commit it on friday. Gerd GerdP wrote
Hi Dave,
the patch changes the rules in the default style so that exist on primary roads are taken into account. More important for your test case: It also changes the code so that exists on motorway_links to other motorway_links are taken into account. The old code only looked at exits on highway=motorway and highway=trunk, the new code checks these highway types: "motorway", "trunk", "primary", "motorway_link", "trunk_link", "primary_link"
So, when you use the patched binary with your own style, the big change is that the condition (highway=motorway_link | highway=trunk_link) & mkgmap:exit_hint=true & mkgmap:dest_hint=true
is now true, and that's why echotags is executed.
You can double check by adding this rule to your lines file: (mkgmap:exit_hint=* | mkgmap:dest_hint=*) { echotags "hint" }
Test this once with the patched binary and once with the normal one and you should see the difference.
Gerd
From:
daveswarthout@
Date: Tue, 19 May 2015 18:37:26 -0700 To:
mkgmap-dev@.org
Subject: Re: [mkgmap-dev] [Patch v1] improve handling of exits
When you say you didn't change the echotags code, did you rewrite the style rule? If you did that I'm embarrassed to admit I am unable to see the difference. AFAIK I'm using the exact same style file for both runs, but the run with the older executables produced no output. I replaced that jar file with the new one (modified r3598) and all of a sudden output from echotags appeared. Anyway, it's all good.
On Tue, May 19, 2015 at 11:03 AM, GerdP <
gpetermann_muenchen@
> wrote: Hi Dave,
I am not sure if you understand that echotags always worked.
The problem is/was that the rule was not executed because
the expression evaluated to false.
In other words, I did not change the echotags code,
I changed the code so that it is more likely that the
term "mkgmap:exit_hint=true & mkgmap:dest_hint=true"
evaluates to true.
Gerd
Dave Swarthout wrote
On Mon, May 18, 2015 at 7:51 AM, Gerd Petermann <
gpetermann_muenchen@
wrote:
@Dave: Please note the changes in the default style.
It would be great if you could test this patch and maybe suggest
a better description of the two options.
Sorry for the delay in responding. The modified r3598 produced output from
the echotags function. I did not see anything regarding the exit_to tags
but my test destination tag was processed and this output went to stderr:
4611686018427392234 (168231839) -
[highway=motorway_link,destination=Beltline Road
East,mkgmap:exit_hint=true,mkgmap:dest_hint=true,bicycle=yes,mkgmap:way-has-pois=true,mkgmap:exit_hint_ref=195A,mkgmap:label:1=Dest:
Beltline Road East,oneway=yes] before
4611686018427392234 (168231839) -
[highway=motorway_link,destination=Beltline Road
East,mkgmap:exit_hint=true,mkgmap:dest_hint=true,bicycle=yes,mkgmap:way-has-pois=true,mkgmap:exit_hint_ref=195A,mkgmap:label:1=Dest:
Beltline Road East,oneway=yes] after
*(highway=motorway_link | highway=trunk_link) & mkgmap:exit_hint=true &*
*mkgmap:dest_hint=true*
* { echotags "before";*
*name '${destination:ref|subst: =>} ${destination|subst:;=> |subst:/=> }'
|*
* '${ref|subst: =>} ${destination|subst:;=> |subst:/=> }' |*
* '${destination|subst:;=> |subst:/=> }' |*
* 'Exit ${mkgmap:exit_hint_ref} ${mkgmap:exit_hint_name}' |*
* 'Exit ${mkgmap:exit_hint_ref} ${mkgmap:exit_hint_exit_to}' |*
* 'Exit ${mkgmap:exit_hint_exit_to}' |*
* 'Exit ${mkgmap:exit_hint_name}' |*
* 'Exit ${mkgmap:exit_hint_ref}' ;*
*echotags "after"*
* }*
This is my rule for debugging the destination tag and it appears before
the
rule above:
*destination=* & (highway=motorway_link | highway=trunk_link) {name 'Dest:
${destination}' }*
My exit_to rule is for points is here:
*exit_to=* {name 'Exit ${ref} ${exit_to}' | '${exit_to}' } [0x12615
resolution 24]*
This produces the desired output on my maps but doesn't show up in stderr
because I haven't yet put echotags into my points style file.
Thanks for the good work....
--
Dave Swarthout
Homer, Alaska
Chiang Mai, Thailand
Travel Blog at http://dswarthout.blogspot.com
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@.org
--
View this message in context: http://gis.19327.n5.nabble.com/Patch-v1-improve-handling-of-exits-tp5845035p...
Sent from the Mkgmap Development mailing list archive at Nabble.com.
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev@.org
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
-- Dave Swarthout Homer, Alaska Chiang Mai, Thailand Travel Blog at http://dswarthout.blogspot.com
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev _______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
-- View this message in context: http://gis.19327.n5.nabble.com/Patch-v1-improve-handling-of-exits-tp5845035p... Sent from the Mkgmap Development mailing list archive at Nabble.com.