DD4hep Meeting November 7th, 2024
Present: A.Sailer, M.Frank, P.McKeown, S.Banerjee
Apologies:
Link to minutes of previous meeting
Merged PRs
- PR 1326: Add xercesc parsing of environment variables in the xml path
Tricky how variables are parsed: environment, expression evaluator etc.
- PR 1327: ASSIMPWriter.cpp: fix "warning: unqualified call to 'std::move'" (Valentin)
- ISSUE 1228:
PR 1329: Edm4hep format as default when both edm4hep and lcio enabled (Brieuc Francois)
- PR 1331: conditions: use static unsigned long long int (Andre).
Causes failutres in latest ROOT release.
- PR 1332: test_import: change error to fehler to avoid triggering test failure (Andre)
- PR 1333: Use edm4hep::labels::CellIDEncoding instead of "CellIDEncoding" (Juan Miguel Carceller)
But: Currently not compiling on LCG105a
- PR 1334: Ci fixes (Andre)
- ISSUE 1330: Kill (and nicely escape) ddsim applications with CTRL-C
PR 1336: Add implementation for signal handling. Use in DDG4 (Markus)
- PR 1337: CMake: explicitly ask for ROOTEve component (Andre)
- PR 1338: add a MacOS CI build again, based on BREW (Andre)
- PR 1339: Fix compilation error on MACOS (Markus)
Corollary of PR 1336
- PR 1341: Add test using signal handler from DDCore (Markus)
Corollary of PR 1336
Pending Problems
- ISSUE1199: Persist hepmc vertex status code into output
==> Should we eliminate Geant4EventReaderHepMC
==> Decision: Geant4EventReaderHepMC will be removed
- ISSUE1285: Memory Consumption of Sensitive Detectors
==> Should be handled as illustrated in PR1288. Intrinsic problem of the Geant4VolumeManager;
- ISSUE1296: DetectorImp::endDocument() called twice for some geometries?
==> Unsure if it is a problem wrongly configuring multiple xml inputs.
- ISSUE1297: Possible to dump all available readout elements?
==> Possibly a copy of unsolved ISSUE580 from Valentin about enumerating all sensitive cells.
- ISSUE1319: Problem with regexSensitiveDetector and volume IDs
==> Interesting use case for reg-ex sensitive detectors
having a VolumeMgr equivalent.
- ISSUE1320: Replacing a Handle<Segmentation> which contains a MultiSegmentation with the underlying Segmentation
===> Unclear what he really means
Pending:
- PR1240: Add multithreading support to ddsim
- PR1304: SurfaceManager: lazily construct surface maps on demand (Leonhard Reichenbach)
- PR1314: Cylinder segment surface (Armin Ilg)
- ISSUE 1335: Uniform random picking of points on detector surfaces for noise studies
This problem is highly linked to #1297 and #580 .
A even only half way efficient generic solution is close to impossible to implement.
Possible solutions are only possible if a segmentation allows to enumerate easily.
- ISSUE 1340: Not saving contributions for calorimeter hits.
Write out a root file from ddsim removing all contributions collections.
Closed issues and fixed problems
- ISSUE1322: materialScan on CAD-imported geometry : strange results...
==> Issue of TGeoTessellated shapes. They do not participate in tracking.
Needs fixes in TGeo, but will not happen soon.
- ISSUE 1323: Instalation CRY libraries
Round Table
Andre: NTR
Markus: NTR
Peter: NTR
Sunnar: CMS using 1.27 - no issues to report
AOB
-----
Next meeting: November 7th, 2024
There are minutes attached to this event.
Show them.