Feature celeritas integration #1370
Conversation
DDG4/src/Celeritas.cpp
Outdated
| // Set along-step factory | ||
| so.make_along_step = celeritas::UniformAlongStepFactory(); |
There was a problem hiding this comment.
This indicates a "zero" magnetic field: is there a way to assert that DD4HEP doesn't have a mag field definition so that it's consistent?
DDG4/src/Celeritas.cpp
Outdated
| // Use Celeritas "hit processor" to call back to Geant4 SDs. | ||
| so.sd.enabled = false; |
There was a problem hiding this comment.
We definitely shouldn't have this. Where were these lines copied from? We should probably use something closer to https://gitlab.cern.ch/bmorgan/athena/-/blob/6072d57711365aa3a58c38ff1134e3f0c569c78b/Simulation/G4Utilities/G4UserActions/src/Celeritas.cxx
There was a problem hiding this comment.
I think that's where I took it from. Am i looking in the wrong branch? https://gitlab.cern.ch/bmorgan/athena/-/blob/integrate-celeritas/Simulation/G4Utilities/G4UserActions/src/Celeritas.cxx?ref_type=heads&blame=1
Co-authored-by: Seth R. Johnson <[email protected]>
…on definitions placed outside the dd4hep::sim namespace
Test Results 14 files 14 suites 7h 10m 57s ⏱️ Results for commit 5ec880e. ♻️ This comment has been updated with latest results. |
|
Hi @rahmans1 Thank you for the interesting feature. Please format the original PR message with BEGIN/ENDRELEASENOTES. |
BEGINRELEASENOTES
Integrate track offloading to celeritas (https://celeritas-project.github.io/celeritas)
ENDRELEASENOTES