Benjamin Michael Wynne authored 4 years ago and Edward Moyse committed 4 years ago

Configure PixelTDAQCondAlg with empty read key when offline. Remove hack dealing with the old dangling dependency

The hard-scatter and background `McEventCollection` objects are merged in Overlay jobs.
This merging has to occur before the collection is accessed via the `HepMcParticleLink`
objects owned by ID and Muon SimHits. As the dependency is not explicit it is not picked
up by the MT scheduler. Therefore the dependency has to be added manually to affected Algorithms.
This is the same fix as !30525, but for the old-style rather than CA-based configuration.

 - exec
 - print function

Set the Cardinality of ID and MS digitization Algorithms in Overlay
jobs to be equal to the number of threads.

John Derek Chapman authored 5 years ago and Walter Lampl committed 5 years ago

This commit adds an `m_doNoise` configurable property to the
`FrontEndSimTool` base class. This is used to determine whether
or not the `ThermalNoise(...)` and `RandomNoise(...)` functions
should be called in the derived classes. This should replicate
the behaviour in `21.0` where the `PixelNoisyCellGenerator` is
only used when `digitizationFlags.doInDetNoise()==True`.

Former-commit-id: 5a775ae0

Soshi Tsuno authored 6 years ago and Walter Lampl committed 6 years ago

Former-commit-id: 27eaffd5

Former-commit-id: 04d0b0124ffbefc5b42290961365e64db61926de

Former-commit-id: d013975e

Former-commit-id: d1854ec4

The common geometry configuration stored in the Oracle table
"AtlasCommon" can only be accessed via InDetGMJobProperties as it
is implemented as properties of GeometryFlags which is member of
InDetGMFlags. Because the properties are (mostly) general (like
the run period) they should not be part of the ID only, but really
"common". It is already bad enough that the only specific field
in the table is called "STRIPGEOTYPE" ... On top, it turned out
that InDetGMJobProperties.py and LArGMJobProperties.py are quite
carbon copies of each other. These flaws are addressed in the
implementation of CommonGMJobProperties.py which provides access
to the common properties. Secondly it serves as base-class for the
classes in InDetGMJobProperties.py/LArGMJobProperties.py to reduce
code duplication. The modification is purely technical and should
not change anything. It passes the Tier0 tests. All clients should
be adjusted, but it was not possible to find test jobs for all of
them.


Former-commit-id: c79918fa

This reverts commit cdf718bf [formerly 9d67f5b8].


Former-commit-id: 008b62d0

This reverts commit 9156e32a [formerly f73d17d3], reversing
changes made to 27d106d1 [formerly 8f2f5188].


Former-commit-id: 9d67f5b8

Former-commit-id: 1ee19443

Former-commit-id: 37398d20

Former-commit-id: 166ef32d

Resolved the mmory issue (one of the containers, which would normally be cleared by going out of scope, had to be cleared manually). Still a difference in the output but I think that again this is becuase of the random number generaters. Will be debugging against old commit to check.


Former-commit-id: 3ff21edc

Former-commit-id: 0a0c4caa

Although there were no differences in the file outptu of the last commit, the run-time has slowed right down. I'm going to have to start over as it's not clear from debugging where the lag is coming from. I suspect its the EnergyDepSvc. Starting from a previouys commit and combining planar and DBM.


Former-commit-id: 429273c9

Former-commit-id: d06b0088

A new digiSteeringConf is required to configure digitization to write out SDOs
linked to Geantinos when in-time Geantino pile-up truth is saved. Fixes ATLASSIM-3193


Former-commit-id: 38915bd5