Rachid Mazini authored 3 years ago and Atlas Nightlybuild committed 3 years ago

Switch from shared to conventional interpolation class pointers

See merge request atlas/athena!51501

(cherry picked from commit 7bc528a1abe0b5c9dd08e7691739f5349243331f)

1af72d44 Switch from shared to conventional interpolation class pointers

Mostly trimming long lines and some general readability improvements.

- limit line length where sensible
- covert double-underscore names to single-underscore
- shorten some of the excessively long variable names
- move imports to top of file where possible

Schedule EventInfoCnvAlg only if it has not yet been scheduled

Use `ConfigFlags` instead of `TriggerFlags` to configure EDM.

Missing override keywords.

Add helper functions to `Athena::SealDebug` to enable/disable core
files (equivalent to what can be achived by `ulimit -Sc` in bash). This
can be useful in code to diagnose rare crashes but without enabling core
files for the entire application to avoid filling up local disks in case
of unrelated crashes.

Work around cppcheck false positive.

Add `DumpCoreFile` property to try to produce a system core dump file as
a last action in the signal handling. Whether this works or not will
depend on the resource limits (e.g. `ulimit -c`).

Previously it was necessary to enable the "Seal" signal handler (e.g.
via `FatalHandler=639`) to get core files produced.

Fix typo in variable name.

- remove temporary FATAL ignore pattern (ATEAM-731)
- explicitly ignore lines with memory addresses

The RNGSeeding test takes way to long to run in the dbg build (~1800s).
Just disable it in that case.

`regHandle` is already called by the algorithm base class.