[mkgmap-dev] Work on is_in branch

Wed Feb 12 15:47:18 GMT 2020

Hi Gerd

The re-structuring makes this difficult.

I propose a function in IsInUtilTest with the same interface as
calcInsideness from IsInUtil that somehow drives the real function IsIn
Function to collect and build the IN/ON/OUT intflag.

Ticker

On Wed, 2020-02-12 at 15:28 +0000, Gerd Petermann wrote:
> Hi Ticker,
> 
> did you run the unit tests? This should download a newer version of
> the samples.
> 
> Gerd
> 
> ________________________________________
> Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag
> von Ticker Berkin <rwb-mkgmap at jagit.co.uk>
> Gesendet: Mittwoch, 12. Februar 2020 16:23
> An: Development list for mkgmap
> Betreff: Re: [mkgmap-dev] Work on is_in branch
> 
> Hi Gerd
> 
> Here it is - changes are:
> 
> - Some restructuring with early stopping where possible.
> 
> - Merging polygons for POINT IN/ON test so a point on shared boundary
> becomes IN rather than ON.
> 
> - Not merging polygons when no need.
> 
> - Make the function cacheable, so that there is negligible cost to
> the
> second call:
> highway=path & is_in(landuse, residential, all)=true [0xAA]
> highway=path & is_in(landuse, residential, all)=false [0xBB]
> 
> - Improved the layout of documentation in the Style Manual.
> 
> - Fixed quite a few problems.
> 
> I've left quite a lot of debug in for the moment, I think there will
> still be work to do.
> 
> It gives correct answers to 'point-on.osm'. I haven't worked through
> is
> -in-hook-sample-v3.osm yet because I wanted to get this revision out
> to
> replace faults in the previous versions.
> 
> Ticker
> 
> On Tue, 2020-02-11 at 15:49 +0000, Gerd Petermann wrote:
> > Hi Ticker,
> > 
> > whatever you plan to do. I moved the code to the lib because it is
> > easier to write a unit test.
> > I would not invest much time to avoid a few tests which only happen
> > in very rare cases. Makes testing more complicated and code less
> > readable.
> > 
> > Gerd