Oana Vickey Boeriu authored 4 years ago and Atlas Nightlybuild committed 4 years ago

Set container splitlevel 1 for DAOD PHYSLITE

See merge request atlas/athena!36372

(cherry picked from commit c790ca4bdca5b36fc4e4ae5013225f42f6d61938)

a33831bb set container splitlevel 1 for phys/physlite
297597f1 only physlite
438e5cbd set to splitlevel 99

Remove unused and undefined COOL_LIBRARIES.

Frank Berghaus authored 4 years ago and Adam Edward Barton committed 4 years ago

This has just laid dormant ever since the CMT->CMake migration. :-(

At the same time introduced some tests for the implicit copy-constructor
and copy-operator of xAOD::EventFormat.

Cleanup cmake configuration, enable flake8 and make code compliant.

This reverts merge request !35330

No need to create a subprocess to find the current directory!

Alaettin Serhan Mete authored 4 years ago and Adam Edward Barton committed 4 years ago

OutputStreamAthenaPool + PrimaryDPDMaker + LongLivedParticleDPDMaker: Python3 fixes, see ATLASRECTS-5598

Add abstract method IAthenaOutputTool::preStream().
To be called just before streaming objects.

Marcin Nowak authored 4 years ago and Adam Edward Barton committed 4 years ago

by accessing MetaDataSvc instead of MetaDataStore
Also added MetaDataSvc::record(unique_ptr, key) method

Add registerTrigNavThinningSvc(), to allow setting up TrigNavigation thinning
from TriggeJobOpts (before streams are created).

Enable MT thinning in OutputStreamConfig.

Add options to output stream configuration for a TrigNavigation
thinning service.

A Splitlevel of 0 is found to result in slightly smaller derivations
(couple %), which is also faster to read.
More details:
https://cds.cern.ch/record/2654407
page 9+

Oana Vickey Boeriu authored 5 years ago and Atlas Nightlybuild committed 5 years ago

Double autoflush for DAODs

See merge request atlas/athena!31063

(cherry picked from commit 0ff9af10b9c97f475547e8f3cbd75bebbf18ee42)

111936a4 Double autoflush for DAODs

Pending thread-safety checker changes warn about potentially unsafe
copies of coral::AttributeList.  Avoid that here.

Don't hide errors if importing AnalysisToolsConf fails.

- has_key -> in

Rename OutputStreamAthenaPool.OutputStreamAthenaPool module to OutputStreamAthenaPool.CreateOutputStreams

Having a module with the same name as the package name causes problems with syntax like:

```from OutputStreamAthenaPool.OutputStreamAthenaPoolConf import ...```

in python 2.7, so the module has been renamed to `CreateOutputStreams` and all references in the code
have been adapted.

Associate a ThinningCacheTool with each output stream, in order to support MT thinning.

The changes belong to a large campaign aimed to clean up Athena code from incidents wherever possible.
These include:
  * Dropped redundant includes to FileIncident.h
  * Cleaned up all references to the UpdateOutputFile, BeginFile and EndFile incidents which are never fired
  * DecisionSvc cleaned up from the redundant inheritance from IIncidentListener
  * ByteStreamEventStorageInputSvc cleaned up from the redundant private service handle to IncidentSvc

scheduled to be written and not yet available in StoreGate.

Peter Van Gemmeren authored 5 years ago and Adam Edward Barton committed 5 years ago

Add seperate property to allow setting of key for old EventInfo, to avoid having to modify many clients.

Updated `OutputStreamConfig` module to enable writing `EventStreamInfo` and `IOVMetaDataContainer`
objects to the output file. Some IOVMetaDataContainer objects are still not copied from input to
output file, so there is still work to be done. This change at least allows the outputs of jobs
configured with `ComponentAccumulator` to be read by diff-root and therefore allow the outputs
to be compared with jobs configured with the old configuration code.

Fix warnings/errors flagged by `flake8`. Mostly unused variables/imports
or stylistic code problems.

Jack Cranshaw authored 5 years ago and Walter Lampl committed 5 years ago

list.