sed-eew / sed-eew-seiscomp-contributions Goto Github PK

In order to compile this along with SC 4.x you should remove scvsmag from branch v4. Otherwise the target scvsmag is defined twice.

Shouldn't the sort command below be used with the -g or -n option for numerical sorting?

https://github.com/SED-EEW/SED-EEW-SeisComP-contributions/blob/3c7941f785ead7b7f3148b93cbdce9060ae99e5a/apps/eew/scvsmag/timeline.cpp#L140C34-L140C34

2023-07-20 21:05:44 INFO seiscomp/src/extras/sed-eew/apps/eew/scvsmag/timeline.cpp: In member function ‘bool Seiscomp::Timeline::feed(const Seiscomp::DataModel::VS::Envelope*)’:
seiscomp/src/extras/sed-eew/apps/eew/scvsmag/timeline.cpp:140:25: error: ‘gainUnit’ was not declared in this scope
  140 |                         gainUnit = stream->gainUnit();
      |                         ^~~~~~~~

Add a type to gainUnit for correct declarations, at least auto. Please test compilation prior to committing your changes otherwise the builds on our end will fail.

When we tried to compile on Debian 11 using libboost 1.74.0 we had next errors:

sceewenv/main.cpp:144:85: error: '_4' was not declared in this scope
144 | _eewProc.setEnvelopeCallback(boost::bind(&App::handleEnvelope, this, _1, _2, _3, _4));
sceewenv/main.cpp:144:81: error: '_3' was not declared in this scope
144 | _eewProc.setEnvelopeCallback(boost::bind(&App::handleEnvelope, this, _1, _2, _3, _4));
....

We added
#include <boost/bind/placeholders.hpp>
using namespace boost::placeholders;
To:
sed-addons/apps/eew/scfinder/main.cpp
sed-addons/test/eewamps/testprocs.cpp
sed-addons/test/eewamps/testgba.cpp
sed-addons/test/eewamps/testgba.cpp

After modifications it worked well.

We are using 5.1.1 tag

1. Implement eewenv.debug.skipDataOlderThan = none and r eewenv.debug.maxDelay = none in order to allow envelope post processing without config change.

2. Allow --playback in scfinder to switch these none parameters in order to allow finder playback without config change.

Please remove scvsmag from the v4 branch otherwise we cannot build the package for SeisComP 4.x.

The link for FinDer library is broken. Can you fix this?

@billyburgoa

In sceewlog, we can't use SeisComP variables (e.g. @datadir@) in file path configuration although suggested in doc.

@camilo @billyburgoa

I can't find how to install "The SED SeisComP contributions data model extension" could someone help me how to do it? I find it a big obstacle for the installation