Opened 11 years ago
Closed 6 years ago
#5140 closed enhancement (wontfix)
KML with space around tuples separator considered invalid
Reported by: | strk | Owned by: | warmerdam |
---|---|---|---|
Priority: | low | Milestone: | |
Component: | OGR_SF | Version: | unspecified |
Severity: | normal | Keywords: | |
Cc: | Kyle Shannon |
Description
I've been told ogr2ogr is unable to read this KML: http://firms.modaps.eosdis.nasa.gov/active_fire/kml/Europe_24h.kml
Full discussion about this from the PostGIS side (which was also bogus, and now fixed): http://trac.osgeo.org/postgis/ticket/2372
Note that according to the NASA data download site the KML is perfectly readable with google earth: http://earthdata.nasa.gov/data/near-real-time-data/firms/active-fire-data#tab-content-2
Change History (8)
comment:1 by , 11 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
comment:2 by , 11 years ago
One could expect Google software to be used as "reference implementation" for reading Google formats, don't you ? In other words, do you prefer GDAL to be unable to read a file that Google Earth can read ? As a user, I'd feel inclined to consider GDAL (and PostGIS) "bogus" (for right or wrong). How about just raising a WARNING ?
Be conservative in what you do, be liberal in what you accept from others ... ref: http://en.wikipedia.org/wiki/Robustness_principle
Maybe not to be called "defect" but "enhancement" ?
comment:3 by , 11 years ago
Priority: | normal → low |
---|---|
Resolution: | wontfix |
Status: | closed → reopened |
Type: | defect → enhancement |
The "old" KML driver is more or less in low maintenance mode, but if someone wants to take a crack at that, without breaking the parsing of <coordinates> with several tuples, but I'd be with Olivier that it is a slippery road...
The new LIBKML one based on libkml seems to be OK with spaces around the comma for a Point (the parsing of coordinates is done by libkml itself). As far as the UTF-8 issue is concerned, not much can be done unfortunatelly.
comment:4 by , 11 years ago
Cc: | added |
---|
comment:5 by , 11 years ago
Sandro, you may have a look at #4347 where it is stated that versions of Google Earth >= 6.1 no longer handle coordinates with spaces before/after commas. So I suspect the version of Google Earth that was used to read Europe_24h.kml is older than 6.1. In which case, we should really close this ticket as wontfix.
comment:6 by , 11 years ago
Even, I just tried reading it with Google Earth 7.1 and 7.0, and it worked. I suspect that they broke a lot of kmls and went back to the lenient parsing. I don't have a 6.x to test on though.
comment:7 by , 6 years ago
Google Earth 7.3.0 opens faulty KML files. Here is another reference which states that spaces are not allowed, the OGC KML 2.3 standard http://docs.opengeospatial.org/is/12-007r2/12-007r2.html
A singlespace separates the tuples and commas separate the ordinates.
Reading libraries could be lenient but based on the missing activity here and on a mailing list archive review this is probably not a big problem for the users.
comment:8 by , 6 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
If someone wants to fix that, they could propose a patch to libkml https://github.com/libkml/libkml
The file is broken in many aspects.
It advertizes UTF-8 but has non UTF-8 characters, which cause the Expat parser to stop the parsing.
And as far as spaces are concerned, according to https://developers.google.com/kml/documentation/kmlreference?hl=fr#point, "Do not include spaces between the three values that describe a coordinate."
One could expect NASA to be able to produce valid KML files...
So I would consider this as a "wontfix"