Estimate `MRelative` based on NetworkMagnitude

damb commented 2 years ago

@mmesim, here a draft implementation for #118.

Features and Changes:

By default estimate MRelative based on the preferred DataModel::Event magnitude i.e. the NetworkMagnitude. Alternatively, the estimation may be still performed based on the StationMagnitude. Note that this requires both the Event and the Origin including the preferred magnitude to be available within the EventParameters.
Whether to use the NetworkMagnitude or the StationMagnitude is configurable by means of the bindings configuration (i.e. on sensor location granularity).

Closes #118.

mmesim commented 2 years ago

👍 I think it is ok.

mmesim commented 2 years ago

@damb Question: When I compile this branch I still get the bug regarding phase association. Do you think it is necessary to remove it?

damb commented 2 years ago

I rebased the branch. The most recent changes from master should be available, now.

mmesim commented 2 years ago

@damb Vielen Dank! ❤️‍🔥

damb commented 2 years ago

@mmesim, I don't plan to make further changes here. Feel free to rebase/edit this feature branch.

swiss-seismological-service / scdetect