[Octave-bug-tracker] [bug #58854] [octave forge] (mapping) kmlread does not handle id tags when trying to parse placemark text

[Philip Nienhuis]

Follow-up Comment #12, bug #58854 (project octave):

That Nan thing is a very old bug :-) thx forspotting that. I'll fix it.

<altitudeMode>clampToGround</altitudeMode> doesn't quite rule out elevation
data. The elevations are behind the Lat/Lon coordinates, see e.g., first
Placemark in ParraWirra.kml:

:
          <Placemark>
            <TimeSpan>
              <begin>2010-01-01T00:00:00Z</begin>
              <end>2010-01-01T00:00:00Z</end>
            </TimeSpan>
            <Style>
              <IconStyle>
              <scale>0.9</scale>
              <Icon>
               
<href>http://connect.garmin.com/image/main/icons/kml/grn-play.png</href>
              </Icon>
            </IconStyle>
            </Style>
            <Point>
              <altitudeMode>clampToGround</altitudeMode>
             
<coordinates>138.83341203443706,-34.710941025987267,346</coordinates>
            </Point>
          </Placemark>
:


The last ",346" in the coordinates subnode is the elevation. I think the
"clamptoground" option usually does need an elevation as the processing
application undoubtedly needs to know the ground level elevation to clamp to
in the first place, doesn't it.
But yes in COURSE_20667693.kml they're lacking. I guess to cater for that
requires testing for nr. of commas (field separators) in the coordinates
subnode and based on that perform the reshape and Z assignment.
Needs to be done elsewhere in kmlread.m as well, I'm afraid. Could be a bit of
work.

As I said in comment #6 I'll put this aside for the next period; if you can
suggest fixes or upload patches I'll happily review them.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?58854>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/