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