DD4hep Developers Meeting

Europe/Zurich
zoom

zoom

Andre Sailer (CERN), Markus Frank (CERN)
Description

Please Join DD4hep developer's meeting via Zoom

Zoom Meeting ID
68917133930
Host
Andre Sailer
Useful links
Join via phone
Zoom URL

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.
    • 15:30 15:50
      Around the table 20m