<div dir="auto">Well I would like it to apply to non routable lines too - if continue with_actions is used - basically just treat routable and non routable lines the same (the initial check should only look at routable lines though I guess).<div dir="auto"><br></div><div dir="auto">Before 2707 if I remember right all copies were removed no matter if you used continue, continue with_actions or no continue.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sep 19, 2017 10:07 AM, "Gerd Petermann" <<a href="mailto:GPetermann_muenchen@hotmail.com">GPetermann_muenchen@hotmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Feix,<br>
again: There REALLY is no problem with continue or continue with_actions here and there never was. The function triggered by the tag just doesn't do what you expect.<br>
<br>
I try again to make clear how mkgmap:set_unconnected_type works:<br>
The code in mkgmap keeps routable lines separated from non-routable lines. Each line is based on a way that was processed by the style rules in the lines file.<br>
If the style adds e.g. 4 different lines for the same OSM way you have 4 line instances refering to the same OSM id, each may be routable or not.<br>
After (!) all OSM elements are processed by the style rules mkgmap computes the nodes which are shared by routable lines (or short roads).<br>
If a road is not connected to any other road (one with a different OSM id) and the tag mkgmap:set_unconnected_type=<wbr>none<br>
is set this routable line is not added to the map. The line instance is removed. In r2599 this also triggered the removal of all non-routable lines for the same OSM way.<br>
This triggering was removed with r2707 so that now the overlay line is still added to the map.<br>
<br>
My understanding is that you want to control this triggering somehow and we have to think about a situation where the style adds e.g. 2 roads and one overlay line<br>
for the same OSM way and only one road has the tag mkgmap:set_unconnected_type=<wbr>none.<br>
<br>
Gerd<br>
<br>
<br>
<br>
______________________________<wbr>__________<br>
Von: mkgmap-dev <<a href="mailto:mkgmap-dev-bounces@lists.mkgmap.org.uk">mkgmap-dev-bounces@lists.<wbr>mkgmap.org.uk</a>> im Auftrag von Felix Hartmann <<a href="mailto:extremecarver@gmail.com">extremecarver@gmail.com</a>><br>
Gesendet: Dienstag, 19. September 2017 09:29:09<br>
An: Development list for mkgmap<br>
Betreff: Re: [mkgmap-dev] mkgmap:set_unconnected_type differentiate between connected on both sides or on one side only<br>
<br>
oh yeah - I remember.<br>
<br>
The problem ist neither the old nor the current behaviour works correctly in comparison to other tags.<br>
I complained that it applies to all occurences - even though using continue<br>
(i only used a line like highway=abc {set mkgmap:set_unconnected_type=<wbr>none} [0x?? road_class=? road_speed=? continue] )<br>
<br>
I really think the best solution if strict sticking to rules without [] block, and respecting continue vs continue with_actions is not possible would be to have to different special tags - one removing also all non routable overlay lines, while the other tag does not.<br>
Back then I only used mkgmap_set_unconnected_type=<wbr>none to remove unconnected roads that make autorouting go crazy (it's unconnected - someone starts a route there - and the route just breaks - mainly because people did not connect the road correctly - and it has nearly invisible gaps on both side - which Potlach v1 was a bit famous for) - now the use case is more to filter out clutter for the semi_connected so I rather want to have it also filter out all overlays. (I'm fine using continue with_actions as supposed).<br>
<br>
On 19 September 2017 at 08:16, Gerd Petermann <<a href="mailto:gpetermann_muenchen@hotmail.com">gpetermann_muenchen@hotmail.<wbr>com</a><mailto:<a href="mailto:gpetermann_muenchen@hotmail.com">gpetermann_<wbr>muenchen@hotmail.com</a>>> wrote:<br>
Well, I just noticed that mkgmap:set_unconnected_type is not yet documented.<br>
When I implemented it with r2599 I decided to remove all overlay lines if<br>
the road is unconnected and mkgmap:set_unconnected_type=<wbr>none is used.<br>
Later, Steve changed this in r2707, the reason was that you complained about<br>
the old behaviour.<br>
Please read these old threads carefully:<br>
<a href="http://gis.19327.n8.nabble.com/Serious-mkgmap-Bug-set-unconnected-type-not-respecting-continue-command-tp5775051.html" rel="noreferrer" target="_blank">http://gis.19327.n8.nabble.<wbr>com/Serious-mkgmap-Bug-set-<wbr>unconnected-type-not-<wbr>respecting-continue-command-<wbr>tp5775051.html</a><br>
<a href="http://gis.19327.n8.nabble.com/Commit-r2707-Don-t-complete-remove-an-unconnected-road-tp5777740.html" rel="noreferrer" target="_blank">http://gis.19327.n8.nabble.<wbr>com/Commit-r2707-Don-t-<wbr>complete-remove-an-<wbr>unconnected-road-tp5777740.<wbr>html</a><br>
<br>
My understanding of these discussions is that we need is an option or tag<br>
that controls what to do with an overlay line when the underlying road was<br>
removed.<br>
<br>
Gerd<br>
<br>
<br>
Felix Hartmann-2 wrote<br>
> But does it mean semi_connected or unconnected? I would like to have both<br>
> as option if possible.<br>
><br>
> On 18 September 2017 at 21:22, Gerd Petermann <<br>
<br>
> gpetermann_muenchen@<br>
<br>
>> wrote:<br>
><br>
>> Hi all,<br>
>><br>
>> what do you think about a tag named mkgmap:remove-if-no-road=yes ?<br>
>> I would add code to check all non-routable lines for this tag.<br>
>> If no routable line is found for the same OSM way the line is removed.<br>
>> This check is performed after the processing of the 2 mkgmap:set_xxx_type<br>
>> tags<br>
>> and maybe other routines which remove routable lines because they are too<br>
>> short<br>
>> etc.<br>
>><br>
>> Gerd<br>
>><br>
>><br>
>> Felix Hartmann-2 wrote<br>
>> > 3. I'm fine with another name - maybe mkgmap:set_semi_connected_<br>
>> line=none<br>
>> > and mkgmap:set_unconnected_line=<wbr>none?<br>
>> > Well I actually already went through my style and added<br>
>> > mkgmap:set_semi_connected_<wbr>type=none - I'm just missing the overlays<br>
>> also<br>
>> > being removed as explained to fully use it. I actually had not noticed<br>
>> > that<br>
>> > mkgmap:set_unconnected_type=<wbr>none did not remove all the lines that I<br>
>> > intended to be removed by it because I never properly checked it using<br>
>> a<br>
>> > test file - and there are not many unconnected ways in real OSM data<br>
>> > (while<br>
>> > there are far more semi_connected lines/ways). Actually I think it<br>
>> should<br>
>> > be semiconnected as we alo use unconnected and not un_connected (even<br>
>> > though it is two words in proper spelling).<br>
>><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Sent from:<br>
>> <a href="http://gis.19327.n8.nabble.com/Mkgmap-Development-f5324443.html" rel="noreferrer" target="_blank">http://gis.19327.n8.nabble.<wbr>com/Mkgmap-Development-<wbr>f5324443.html</a><br>
>> ______________________________<wbr>_________________<br>
>> mkgmap-dev mailing list<br>
>><br>
<br>
> mkgmap-dev@.org<br>
<br>
>> <a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer" target="_blank">http://www.mkgmap.org.uk/<wbr>mailman/listinfo/mkgmap-dev</a><br>
>><br>
><br>
><br>
><br>
> --<br>
> Felix Hartman - Openmtbmap.org & VeloMap.org<br>
> Schusterbergweg 32/8<br>
> 6020 Innsbruck<br>
> Austria - Österreich<br>
><br>
> ______________________________<wbr>_________________<br>
> mkgmap-dev mailing list<br>
<br>
> mkgmap-dev@.org<br>
<br>
> <a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer" target="_blank">http://www.mkgmap.org.uk/<wbr>mailman/listinfo/mkgmap-dev</a><br>
<br>
<br>
<br>
<br>
<br>
--<br>
Sent from: <a href="http://gis.19327.n8.nabble.com/Mkgmap-Development-f5324443.html" rel="noreferrer" target="_blank">http://gis.19327.n8.nabble.<wbr>com/Mkgmap-Development-<wbr>f5324443.html</a><br>
______________________________<wbr>_________________<br>
mkgmap-dev mailing list<br>
<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a><wbr><mailto:<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.<wbr>mkgmap.org.uk</a>><br>
<a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer" target="_blank">http://www.mkgmap.org.uk/<wbr>mailman/listinfo/mkgmap-dev</a><br>
<br>
<br>
<br>
--<br>
Felix Hartman - Openmtbmap.org & VeloMap.org<br>
Schusterbergweg 32/8<br>
6020 Innsbruck<br>
Austria - Österreich<br>
______________________________<wbr>_________________<br>
mkgmap-dev mailing list<br>
<a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a><br>
<a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev" rel="noreferrer" target="_blank">http://www.mkgmap.org.uk/<wbr>mailman/listinfo/mkgmap-dev</a><br>
</blockquote></div></div>