Avoid spurious object copy in range for.

This code was used during Run-1 to store HLT rates in COOL.

Due to thread-safety changes in ByteStreamCnvSvc, there is no longer
a m_rawEventWrite member.  Use the interfaces from the base class instead.

Mark Stockton authored 4 years ago and Walter Lampl committed 4 years ago

Requires update Cost workflow for Trig_reco_tf (ATR-20974)
- Define COST BS output as DRAW_TRIGCOST (rather than a second BS to avoid conflicts in ProdSys)
- Split full BS to fill CostMon stream into the DRAW (before splitting other streams as requested)

- DRAW_TRIGCOST needs registering before can be used in ProdSys
- athena arguments for RAWtoCOST updated, but need further changes when scripts are ready

 - Pass class instances by const reference, not by value.
 - assert() with side effects.
 - unused value.
 

Thread-safety annotations should match between declarations and definitions.

Joerg Stelzer authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

When running from the DB in athenaHLT, the HLTPSK needs to be read from
the database directly, `HLTPrescaleCondAlg.Source` needs to be changed
to `DB` to avoid trying to read the key from COOL.

Relates to ATR-22143.

- try to release early possible locks in tbb::concurrent_unordered_map (see also ATR-22112)
- put clearing and size allocation for output arrays of the eventCache_checkRobListToCache function
  at the begin of the function

Always put ROOT into batch mode to avoid additional GUI threads being
created that could cause problems during forking (ATR-21890).

!36223 introduced a slight change in behavior when reading
`DF_Enabled_ROB_IDs`, i.e. it was always overwriting the list of enabled
ROBs even if a value was set explicitly via job options.

Fix bug when parsing job properties of the type `MyAlg.MyTool.MyProp`.
The client name is in this case `MyAlg.MyTool` and not only `MyAlg`.

Package contains old-style data-scouting example for the muon
calibration stream. If we ever get around to using this online, this
could be revived and updated.

Standalone application to write trigger configuration to COOL/IS is no
longer needed (see ATR-19823, ATR-20641).

Add a new `ITrigEventLoopMgr::prepareForStart` method to the
`HltEventLoopMgr` that is being called by the `Psc` before calling the
start method of the `ApplicationMgr`. This will allow the EventLoopMgr to
setup anything that may be required during `start` (e.g. setting up
a partial EventContext, ATR-21331).

Patch sent by @wiedenma

Make sure getoutput() is in the subprocess module.
We still have a python 2 build, so we want thigs to work with both 
py2 and py3.

Fix or remove `atlas_install_*` statements that do not reference any
files.

Make test timeout shorter in opt build.