swiss-seismological-service / scdetect

A computationally efficient earthquake detection module for SeisComP
https://scdetect.readthedocs.io
GNU Affero General Public License v3.0
15 stars 6 forks source link

Compute network magnitudes #106

Closed damb closed 2 years ago

damb commented 2 years ago

Compute network magnitudes as the median of the corresponding station magnitudes.

Closes #103.

damb commented 2 years ago

@mmesim, please check on your side whether both station magnitudes and network magnitudes can be displayed in scolv.

mmesim commented 2 years ago

@luca-s Network Magnitudes are ok Network Magnitudes

Still cannot see Station Magnitudes Screenshot from 2022-05-12 10-17-16

damb commented 2 years ago

@luca-s, do you know which prerequisites need to be fulfilled in order to display station magnitudes in scolv?

DataModel::Magnitude references DataModel::StationMagnitude by means of DataModel::StationMagnitudeContribution already.

damb commented 2 years ago

Ok. Got it.

In order to display DataModel::StationMagnitudes in scolv the WaveformID needs to be set.

EDIT: Actually, I didn't expect this behaviour, due to the member function documentation of DataModel::StationMagnitude::waveformID(): https://github.com/SeisComP/common/blob/a5b9a6ccf3fb93407c2f57df3b068d031f0a16b1/libs/seiscomp/datamodel/stationmagnitude.h#L133-L142

damb commented 2 years ago

@mmesim, could you please check once again on your side?

mmesim commented 2 years ago

Still no station magnitudes.

luca-s commented 2 years ago

@mmesim could you dump the origin in XML so we can see what is missing?

mmesim commented 2 years ago

@damb Great job! It works! @luca-s Thanks a lot for your help!! :tada: :tada: :tada: :tada: :tada: :tada: :tada:

Screenshot from 2022-05-16 17-24-21