mSTS and mMUCH changes from march 2022 runs
Introduces support for mMUCH unpacking in the CRI world + working online monitoring for both systems + various fixes (see commits messages)
Draft status: To be rebased (should be trivial) and merged only after !809 (merged) is merged
Reviewers: @a.toia and @v.singhal (not assigning as both approval needed in my opinion)
Merge request reports
Activity
assigned to @p.-a.loizeau
mentioned in merge request !811 (merged)
added Reconstruction mCBM labels
added 27 commits
-
8d7e7bbc...e13ab847 - 15 commits from branch
computing:master
- 80710bc6 - Update mTOF unpack par class + use 2022 par file to support new modules for runs from 2060 on
- 218b9e13 - Unpacking: add BMon unpacking based on TOF one + BMon/TOF monitoring
- abb05ab1 - [BMon-TOF] Fix time offseting and monitor mapping for Bmon
- bb386b88 - [mBmon] Add Tof unpack par Alias for BMon + related flags and conversions
- 645f74c8 - [mTOF,mBmon] Fixes mBmon and mTOF Buc mapping
- c90285c0 - Add MUCH unpacking
- 63ebf986 - [mSTS] Add new parameter files for 2022
- 43a6a890 - [mSTS][mMUCH] Fixes to get the monitors running
- e9d5bfb3 - [mMUCH] Added Pulser branch support and new Par File with Par File setter in MuchConfig
- 3a1af1fc - [mMUCH] In unp algo, do not print address warning for pulser digis
- 64558c71 - Add missing dict for error msg vector + improve error flag handling in mSTS unp
- 037d7c0f - [mMuch] Fix par file setting in unpack algo-conf + fix par file
Toggle commit list-
8d7e7bbc...e13ab847 - 15 commits from branch
mentioned in merge request !809 (merged)
added 12 commits
- 8ab51f57 - Update mTOF unpack par class + use 2022 par file to support new modules for runs from 2060 on
- 59dad30e - Unpacking: add BMon unpacking based on TOF one + BMon/TOF monitoring
- 2117fdf6 - [BMon-TOF] Fix time offseting and monitor mapping for Bmon
- ebbb46b7 - [mBmon] Add Tof unpack par Alias for BMon + related flags and conversions
- be6f134e - [mTOF,mBmon] Fixes mBmon and mTOF Buc mapping
- 152704cd - Add MUCH unpacking
- 3675ef66 - [mSTS] Add new parameter files for 2022
- 13e80012 - [mSTS][mMUCH] Fixes to get the monitors running
- 72273664 - [mMUCH] Added Pulser branch support and new Par File with Par File setter in MuchConfig
- f4b49aaf - [mMUCH] In unp algo, do not print address warning for pulser digis
- a0bf2937 - Add missing dict for error msg vector + improve error flag handling in mSTS unp
- 334ff079 - [mMuch] Fix par file setting in unpack algo-conf + fix par file
Toggle commit listadded 7 commits
- d4e25ce6 - Add MUCH unpacking
- eecd825e - [mSTS] Add new parameter files for 2022
- 24fd49df - [mSTS][mMUCH] Fixes to get the monitors running
- f7bccabc - [mMUCH] Added Pulser branch support and new Par File with Par File setter in MuchConfig
- 36f2555c - [mMUCH] In unp algo, do not print address warning for pulser digis
- 772b269a - Add missing dict for error msg vector + improve error flag handling in mSTS unp
- acd0f5ea - [mMuch] Fix par file setting in unpack algo-conf + fix par file
Toggle commit list- Resolved by Pierre-Alain Loizeau
Dear @f.uhlig, @v.friese, @p.-a.loizeau, @v.singhal,
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
requested review from @v.singhal
added 8 commits
-
d32603f2 - 1 commit from branch
computing:master
- 66cc14a9 - Add MUCH unpacking
- 410a4145 - [mSTS] Add new parameter files for 2022
- 4767c97e - [mSTS][mMUCH] Fixes to get the monitors running
- 3413aebc - [mMUCH] Added Pulser branch support and new Par File with Par File setter in MuchConfig
- 7331b5af - [mMUCH] In unp algo, do not print address warning for pulser digis
- 4a1cec2c - Add missing dict for error msg vector + improve error flag handling in mSTS unp
- 7d715d9c - [mMuch] Fix par file setting in unpack algo-conf + fix par file
Toggle commit list-
d32603f2 - 1 commit from branch
added 8 commits
-
90ec7a7b - 1 commit from branch
computing:master
- ded606b8 - Add MUCH unpacking
- 6e54816b - [mSTS] Add new parameter files for 2022
- a179c1ec - [mSTS][mMUCH] Fixes to get the monitors running
- 2169676a - [mMUCH] Added Pulser branch support and new Par File with Par File setter in MuchConfig
- 7d85d0f9 - [mMUCH] In unp algo, do not print address warning for pulser digis
- 57e08c82 - Add missing dict for error msg vector + improve error flag handling in mSTS unp
- b48a1031 - [mMuch] Fix par file setting in unpack algo-conf + fix par file
Toggle commit list-
90ec7a7b - 1 commit from branch
added 10 commits
-
b48a1031...66f00fbc - 9 commits from branch
computing:master
- d64b28bf - [mSTS][mMUCH] Fixes to get the evolution plots in the monitors running
-
b48a1031...66f00fbc - 9 commits from branch
enabled an automatic merge when the pipeline for d64b28bf succeeds
The initial commits were included by mistake when merging !811 (merged) without rebasing it interactively first to change the order. The merging was however approved by both detectors responsible persons in the meantime.
So for future reference, all commits tagged with
[STS]
or[MUCH]
in !811 (merged) are described hereThe remaining commit is one which was planned to be added to this MR.