Boost logo

Geometry :

Subject: Re: [geometry] io - WKB
From: Mats Taraldsvik (mats.taraldsvik_at_[hidden])
Date: 2013-08-28 15:52:46


On 08/28/2013 05:37 PM, Barend Gehrels wrote:
> Hi Mats,
> Great, thanks for your efforts!
> On 28-8-2013 12:57, Mateusz Loskot wrote:
>> On 27 August 2013 18:52, Mats Taraldsvik <mats.taraldsvik_at_[hidden]>
>> wrote:
>>> On 07/29/2013 11:44 AM, Mateusz Loskot wrote:
>>>> On 28 July 2013 11:45, Mats Taraldsvik <mats.taraldsvik_at_[hidden]>
>>>> wrote:
>>>>> On 07/18/2013 11:51 PM, Mateusz Loskot wrote:
>>>>>> On 18 July 2013 20:04, Mats Taraldsvik
>>>>>> <mats.taraldsvik_at_[hidden]> wrote:
>>> Sorry for the late response. I finally had time to look at this, and
>>> have
>>> implemented a working MultiPoint-parser. Currently, the implementation
>>> modifies include/boost/geometry/extensions/gis/io/wkb/parser.hpp and
>>> read_wkb.hpp (and also an equals implementation for multipoints in
>>> include/boost/geometry/multi/algorithms/equals.hpp).
>>> Do you want me to move the multipoint implementations to a
>>> multi-directory,
>>> as in the rest of boost geometry?
>>> include/boost/geometry/extensions/multi/gis/io/wkb/parser.hpp and
>>> read_wkb.hpp, perhaps?
>> Yes, this move & include sounds good.
>> Here my old post with outline of the I/O folders structure that you
>> may find useful:
>> I believe it's up to date and we should try to keep it that way.
> Yes, agreed. SVG is released in the meantime.

I have moved the files to adhere to the existing conventions. I have
attached three patches, created from git using this recipe

It would be great if you could review them. If I should submit the
patches using a different approach than this mailing list (trac?),
please tell me and I'll be happy to oblige.

Also, I created a test for LineString, but it did not compile due to a
missing strategy, computing distance. I did not quite understand it, so
I would appreciate if you could look at it as well.

>>> I could only find tests for Point in
>>> extensions/test/gis/io/wkb/read_wkb.cpp, should I add some tests for
>>> LineString, Polygon.. as well?
>> Yes, tests for other geometry types are missing, apparently.
>> If you could add them, that would be awesome.
>>> Should I move my tests for MultiPoint from
>>> extensions/test/gis/io/wkb/read_wkb.cpp to another multi-folder as
>>> well?
>>> extensions/test/multi/gis/io/wkb/read_wkb.cpp?
>> I think we can keep all tests in single place,
>> in extensions/test/gis/io/wkb/ directory,
>> similarly to the current tests for WKT.
> Agreed - for WKB it there is no need to separate multi. I started with
> that in the past and for overlay-algorithms it is certainly useful,
> but for formats it is not really important, or actually inconvenient,
> I consider to move wkt/dsv too.
> Do you have SVN write-access to extensions?
> Regards, Barend


> _______________________________________________
> Geometry mailing list
> Geometry_at_[hidden]

Geometry list run by mateusz at