fix problem with invalid input (ES_WrongFileFormat) caused by
concatenating multiple filenames. athenaHLT.py parses input file
arguments with the 'append' option, so multiple '--file' options need to
be passed.

Our clang builds are currently building against the gcc tdaq builds and
there is not 'clang' platform tag in OKS. Simply revert to a (random)
existing tag to make the SW repository generation work. But this will
obviously not be usable in a partition.

Fixes ATR-22849.

Aleksandra Poreba authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

+ review changes

Remove unused link dependencies, move headers to src directory and
delete obsolete plotting script.

Add the `--loop-files` command line option. When set athenaHLT will loop
over the input files if the maximum number of events is larger than the
number of events in the input files(s). If no event count is given, it
will loop forever.

Also add a new options group for file I/O to make the help message look
a bit cleaner and fix arguments types for number of events.

Produce a "fast" stack trace by default in athena jobs. This initial
stacktrace will only report traces for the current thread but is much
faster than the full trace from gdb (which will follow). This can help in
cases when gdb does not manage to produce a stacktrace (e.g. due to
timeouts as in ATR-22583).

Aleksandra Poreba authored 4 years ago and Frank Winklmeier committed 4 years ago

Including whitespace changes, message formatting

- remove SCHEDULED (prefetched)
- rename DISABLED -> UNDEFINED

Also save robs status with unsigned short to avoid type
misinterpretation issues

The Boost JSON parser, which is used in a number of places, still
uses <boost/bind.hpp> internally. (Instead of <boost/bind/bind.hpp>.)
To silence the messages about that, I added explicit definitions of the
BOOST_BIND_GLOBAL_PLACEHOLDERS macro in all appropriate places.

Aleksandra Poreba authored 4 years ago and Adam Edward Barton committed 4 years ago

Including whitespace changes, message formatting

change of ambiguous names
remove unused variables
simplify the logic
switch to camel case
additional comments/messages to make code more readable

Aleksandra Poreba authored 4 years ago and Walter Lampl committed 4 years ago

Including whitespace changes, message formatting

Spurious copy in range-for.

Remove original attempts for the trigger transform which are no longer needed and could cause confusion

	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/scripts/Trig_tf_main.py
	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/python/trigExe.py
	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/python/trigPostRun.py
	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/python/trigPreRun.py
	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/python/trigUpdateArgs.py

	deleted:    HLT/Trigger/TrigTransforms/TrigTransform/share/Trig_tf_simple.py

Aleksandra Poreba authored 4 years ago and Frank Winklmeier committed 4 years ago

* retrieve Atlas Project name from DB
* move to ROOT 6
* move to cppyy 1.6.1
* whitespace changes to unify document style

* Derive TrigByteStreamCnvSvc from ByteStreamCnvSvcBase instead of ByteStreamCnvSvc.
* Implement explicit EventContext passing through all methods which need it. Make the interface implementation methods retrieve the EventContext and call the context-aware local methods.

Use the new "central" MessageSvc source format. Also move the MessageSvc
initialization after the user job options to enforce our own settings.