Integrate v0 trigger into the online processing chain
This MR integrates the V0 topology trigger into the online processing chain. The chosen method to do used minimal intervention to the existing code base and keeps backwards compatibility with the yaml configuration (EventBuildConfig.yaml). So, it the latter is untouched, the digi multiplicity trigger will be run as before.
To use the V0 trigger, the reconstruction (local reconstruction + tracking) has to be included in the chain. This is not yet automatized. In the configuration file, a node
v0trigger:
max_time_diff: 50 /// time difference between two tracks in a pair
min_z_pca: 1 /// minimum z position of point of closest approach
max_dist_at_pca: 2 /// maximum distance between tracks at point of closest approach
has to added. If this node is found, the V0 trigger with the specified cut values will be applied instead of the digi multiplicity trigger.
In the process of implementing, I noticed that it seems approriate to restructure the entire online code to some extent, since the main steering class Reco becomes more and more spaghetti code. A more modular implementation, separating the trigger from the event builder step, is what I would like to have, but I did not want to introduce major re-arrangements of the code before the main mCBM data taking in May 2024.
EDIT PAL 24/04/2024: retarget to DC branch
EDIT PAL 25/04/2024: retarget to master as not ready to merge in time for DC#5 (will need a further rebase due to this)
Merge request reports
Activity
added Online label
requested review from @j.decuveland
assigned to @fweig
Dear @d.smith, @fweig, @f.uhlig, @v.friese, @p.-a.loizeau,
you have been identified as code owner of at least one file which was changed with this merge request.
Please check the changes and approve them or request changes.
added CodeOwners label
added DC branch label
added 25 commits
-
8de46e4e...a7ab5082 - 21 commits from branch
computing:DC_2404
- 79173df5 - [online] alt fix for FEB index at mSTS chan masks import, closer to original
- 48031ef5 - bugfix: default config for mcbm_qa.C
- e7f38e05 - Integration of V0 Trigger in online chain
- d1b0e75f - Finalize integration
Toggle commit list-
8de46e4e...a7ab5082 - 21 commits from branch
added 7 commits
-
d1b0e75f...c23f0e3d - 3 commits from branch
computing:DC_2404
- 3e2f5fde - [online] alt fix for FEB index at mSTS chan masks import, closer to original
- 798e62b2 - bugfix: default config for mcbm_qa.C
- 75620d3f - Integration of V0 Trigger in online chain
- a3107717 - Finalize integration
Toggle commit list-
d1b0e75f...c23f0e3d - 3 commits from branch
requested review from @fweig and removed review request for @j.decuveland
@fweig Seems Jan is not available for review, so I assign you as reviewer.
removed DC branch label
added 15 commits
-
a3107717...b4e72d23 - 13 commits from branch
computing:master
- ff78c365 - Integration of V0 Trigger in online chain
- 46868e8d - Finalize integration
-
a3107717...b4e72d23 - 13 commits from branch
added 4 commits
-
46868e8d...678b2296 - 2 commits from branch
computing:master
- 90a15ab0 - Integration of V0 Trigger in online chain
- 7e03373a - Finalize integration
-
46868e8d...678b2296 - 2 commits from branch