Draft: Tidy up UT sim conditions for simulation
Corresponds to !578 (merged) which is targeting 2024-patches
.
List of required MRs:
-
!582 (closed) (DD4hep) targeting master
- Simulation conditions made optional by adding the
option='1'
XML attribute.
- Simulation conditions made optional by adding the
-
LHCb!4615 (closed) (DetDesc) targeting master
-
Boole!580 (merged) targeting master
-
lhcb-conddb/lhcb-conditions-database!154: targeting sim10/run3-ideal
- Remove completely
ProductionMap
,PedestalValues
which are not used at all in DD4hep MC - Reduce following conditions inside
NoiseValues
-
SectorNoise
from per strip (.5M entries) to per ASIC (4192 entries) -
cmNoise
from per strip (.5M entries) to per ASIC (4192 entries) -
electronsPerADC
from per strip (.5M entries) to per sector (1048 entries)
-
- Remove completely
-
lhcb-conddb/SIMCOND!240 targeting upgrade/master
- Reduce following conditions in
NoiseValues
-
SectorNoise
from per strip (.5M entries) to per ASIC (4192 entries) -
cmNoise
from per strip (.5M entries) to per ASIC (4192 entries) -
electronsPerADC
from per strip (.5M entries) to per sector (1048 entries)
-
- Reduce following conditions in
Validated by
-
Core Software -
RTA -
Simulation
Merge request reports
Activity
assigned to @hawu
mentioned in merge request !578 (merged)
mentioned in merge request LHCb!4615 (closed)
mentioned in merge request Boole!580 (merged)
mentioned in merge request lhcb-conddb/lhcb-conditions-database!154
mentioned in merge request lhcb-conddb/SIMCOND!240
added UT label
Actually there is already a MR - as far as I know identical to this.
We should not have two MR: once we merge !578 (merged) it will be automatically picked up via the
2024-patches --> master
synchronization.@hawu can you confirm this is identical to !578 (merged)? Then we can close this one.
Yes. This is identical. Let's close then.
However, Boole!580 (merged), lhcb-conddb/lhcb-conditions-database!154, lhcb-conddb/SIMCOND!240 are still needed.
Edited by Hangyi Wu
added Simulation label
The changes included here will be automatically picked up from
2024-patches
tomaster
via !578 (merged), so this MR is no longer needed.