Update the Fatras simulator
Update ActsFatras::Simulator
to use the concrete Fatras event data model. Add a comprehensive integration test that runs the simulation with a large set of different input particles.
Additional changes:
- Rename particle accessors:
.direction()
to.unitDirection()
to clarify that the resulting vector is normalized and.momentum()
to.absMomentum()
to clarify that the resulting value is a scalar and avoid confusion with.momentum()
for track parameters (returns a vector). - Add a
.transverseMomentum()
accessor. - Rename hit accessor:
.direction{Before,After}()
to.unitDirection{Before,After}()
for the same reasons. - The particle id acessor is now
.particleId()
for both hits and particles. - The geometry id accesor is now
.geometryId()
. - Where possible, make hit and particle accessors
constexpr
. - PDG number selectors are now templated on
Acts::PdgParticle
instead ofint
. - Add a
ChargedElectroMagneticPhysicsList
as a pre-defined type. - Add some additional unit vector construction helpers and rename the corresponding header from
Utilities/CurvilinearUnitVectors.hpp
toUtilitities/UnitVectors.hpp
. - Make
GeometryID
,MultiIndex<...>
andBarcode
hashable, i.e. usuable as keys e.g. in astd::unordered_map
.
Depends on !763 (merged) and !768 (merged). Fixes #693 (closed) and #694 (closed).
Edited by Moritz Kiehn
Merge request reports
Activity
Filter activity
changed milestone to %0.17.00
added Component - Fatras Improvement Major labels
added 29 commits
-
69b5eb97...e8ade263 - 13 commits from branch
master
- d385babf - Fatras: use fixed-size array as physics return types
- 77064a0e - Fatras: remove detector parameter from most selectors
- 76d8a91c - Fatras: reorder limit selector params and fix tests
- 777858a6 - Fatras: use std functions for kinematic casts
- cbd239e8 - Fatras: replace SelectorList w/ simpler implementation
- 0d01782e - Fatras: fix process interface and add missing test
- 2ba24d69 - Utilities: rename unit vectors header
- 3dfb3d1a - Utilities: add direction unit vector helper
- c54519d4 - Fatras/Particle: clarify accessor names
- cb9d39c1 - Fatras/Particle: rename particle id accessor
- dd451cdb - Fatras/Particle: add print support
- 1a7badea - Fatras/Hit: clarify accessor name
- b18e018c - Fatras: typedef scattering processes
- 18abbb41 - Fatras: add standard em physics list
- cc06d397 - WIP Fatras: update simulator
- 20b76328 - WIP Fatras: add simulation integration test
Toggle commit list-
69b5eb97...e8ade263 - 13 commits from branch
added 21 commits
-
20b76328...ad6580ff - 9 commits from branch
master
- d6ad870f - Utilities: rename unit vectors header
- 4a068c8b - Utilities: add direction unit vector helper
- 7c1663a8 - Fatras/Particle: clarify accessor names
- 02ea82d7 - Fatras/Particle: rename particle id accessor
- 12d3e204 - Fatras/Particle: add print support
- 446177ab - Fatras/Particle: add copy-setter for particle id change
- 448a9c4e - Fatras/Hit: clarify accessor name
- 1a47df5b - Fatras: small doc word fix
- 077a7fab - Fatras: typedef scattering processes
- 8ed217dc - Fatras: add standard em physics list
- 9980e40c - Fatras: update simulator
- 8759f073 - Fatras: add simulation integration test
Toggle commit list-
20b76328...ad6580ff - 9 commits from branch
assigned to @asalzbur
added 50 commits
-
50feedb9...869395ef - 38 commits from branch
master
- 20af5191 - Utilities: rename unit vectors header
- 4e325cc2 - Utilities: add direction unit vector helper
- 84e44ac5 - Fatras/Particle: clarify accessor names
- a51655a0 - Fatras/Particle: rename particle id accessor
- f4fe77ff - Fatras/Particle: add print support
- 85c5c241 - Fatras/Particle: add copy-setter for particle id change
- 9b83e857 - Fatras/Hit: clarify accessor name
- c24e7f2d - Fatras: small doc word fix
- 220a510b - Fatras: typedef scattering processes
- af992bb1 - Fatras: add standard em physics list
- 20f746a0 - Fatras: update simulator
- e6367380 - Fatras: add simulation integration test
Toggle commit list-
50feedb9...869395ef - 38 commits from branch
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
- Resolved by Fabian Klimpel
Please register or sign in to reply