search

eqcorrscan / EQcorrscan

Earthquake detection and analysis in Python.
https://eqcorrscan.readthedocs.io/en/latest/
Other
166 stars 86 forks source link

Make detection.id XML parsable #201

Closed calum-chamberlain closed 6 years ago

calum-chamberlain commented 6 years ago

Currently Detection.id is a string made up of Template.name and Detection.time, but the default string format for UTCDateTime objects contains colons, which are not parsed by ObsPy when writing QuakeML files. The upshot of this is that, despite the event.resource_id of detected events being set to the Detection.id, if a catalog of detected events is written as QuakeML and read back in, that information is lost.

I would recommend that we change the string formatter for Detection.id so that it changes colons to dashes.

This may require some warnings of compatibility?

calum-chamberlain commented 6 years ago

Note that this was fixed in version 0.2.7.