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

Ander Pijoan ander.pijoan at deusto.es
Thu Aug 1 16:53:45 UTC 2013


-----------------------------------------------------------
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.


Changes
-------

Changed the fixes proposed.


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 (updated)
-----

  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/20130801/b66f6842/attachment.html>


More information about the Marble-devel mailing list