[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 #13, bug #58854 (project octave):

Couldn't resist having a look (turns out my daughter prepares dinner tonight
rather than me so I have a brief spell)

As I wrote over in bug #58701 (gpxread), comment #5:
> mixed time format fields are going to be a big PITA. That is because
Octave's datenum function (like Matlab's) isn't smart enough to parse a date
string vector with varying formats.
... no matter how valid Google declares it to be for kml files :-(

Of course, in theory preparing for different date/time formats in a vector
should be possible but in practice it's going to be a complicated affair
having fragile results at best.
Luckily Google standardized the formats a bit. So checking the lengths using

cellfun ("numel", times);


...followed by filling in missing month/day/hour/minute/second strings with
zeros for each string < 20 chars could be a way out.  However time zone
corrections (the last time format in comment #9) would screw up royally.

So inserting:

      ltime = cellfun ("numel", times);
      if (any (ltime < 20))
        ## Year resolution
        times(ltime == 4) = ...
         cellfun (@(x) [x "-00-00T00:00:00Z"], times(ltime == 4), "uni", 0);
        ## Month resolution
        times(ltime == 7) = ...
         cellfun (@(x) [x "-00T00:00:00Z"], times(ltime == 7), "uni", 0);
        ## Day resolution
        times(ltime == 10) = ...
         cellfun (@(x) [x "T00:00:00Z"], times(ltime == 10), "uni", 0);
      endif


after L.194 solves reading 2020-50km-Course.kml time fields (I just
verified).

As to the missing elevation in COURSE_20667693.kml, that fix was just needed
for Points and involved an easy check on nr. of coordinates.

OK, fixes pushed here:
http://hg.code.sf.net/p/octave/mapping/rev/5325fa8506f7
(I took the liberty of crediting you for your assistance)
and kmlread.m attached, please try, hopefully it works well enough.


(file #49628)
    _______________________________________________________

Additional Item Attachment:

File name: kmlread.m                      Size:8 KB
    <https://file.savannah.gnu.org/file/kmlread.m?file_id=49628>



    _______________________________________________________

Reply to this item at:

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

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