[Marble-devel] Review Request 111810: GeoJSON parser adapted to match the standard.

[Ander Pijoan]

> On Aug. 1, 2013, 3:13 p.m., Bernhard Beschow wrote:
> > First of all, I tested your patch by adopting the vectorosm map theme. It works very well! Here are my comments on your code:

You mean you used the VectorOSM Map theme as it was?

Because I changed this parser because the old one was just to match the tile format that was written in the vectorosm.dgml file which is not exact GeoJSON. With this new parser those tiles ( from osmonimski.ru ) should not work but all the tiles that really follow the GeoJSON format should. So I don't know if you changed the URL in the dgml file where to get the tiles from to try the parser.

My idea now was to make it work with other vector tile servers because I haven't been able to make the vectorosm map theme work. Maybe there have been some changes in the code that need the vector tile managing system to be updeated. But if you say that it has worked with you there might be something I'm doing wrong. I will start with it as soon as the parser is aproved =)


- Ander


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111810/#review36919
-----------------------------------------------------------


On Aug. 1, 2013, 4:53 p.m., Ander Pijoan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111810/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2013, 4:53 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> -Changed Marble's GeoJSON parser (which was a test for just matching Kothic's GeoJSON like format) to match the exact GeoJSON standard : 
> 
> http://www.geojson.org/geojson-spec.html#id3
> 
> -Supported geometry types :
> 
> Point
> MultiPoint
> LineString
> MultiLineString
> Polygon
> MultiPolygon
> 
> -Not supported geometry types:
> 
> GeometryCollection
> 
> -Geometry properties are compared with GeoDataFeature's OsmVisualCategory to see if any of the properties matches a style to give to the geometry. If not it will be shown as a black line.
> 
> 
> Diffs
> -----
> 
>   src/plugins/runner/json/JsonParser.h 4fb072c 
>   src/plugins/runner/json/JsonParser.cpp 86cf66b 
>   src/plugins/runner/json/JsonPlugin.cpp e7ff770 
>   src/plugins/runner/json/JsonRunner.cpp 2098e1f 
> 
> Diff: http://git.reviewboard.kde.org/r/111810/diff/
> 
> 
> Testing
> -------
> 
> Some test files :
> 
> http://tile.openstreetmap.us/vectiles-skeletron/12/1206/1539.json (this file will matches road styles)
> 
> http://tile.openstreetmap.us/vectiles-buildings/12/1206/1539.json (this file matches some other geometry types)
> 
> http://tile.openstreetmap.us/vectiles-water-areas/12/1206/1539.json (this file matches some other multi geometry types)
> 
> 
> Thanks,
> 
> Ander Pijoan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20130824/e4b5b778/attachment.html>