mcbm2024 data recostruction chain:
This merge request makes a small cleaning of the newly introduced reconstruction/qa macros for mcbm2024 data and provides a bash script for a convenient TS reconstruction.
- adding TsEventHeader and the "TimeSlice." branch to the CbmSourceDigiTimeslice class (processed in the run_inspect_digi_timeslice.C macro)
- CbmL1: added the CbmL1:EInitMode flags, the EInitMode::Full is default and provides the initialization of the L1Algo
Merge request reports
Activity
added QA Reconstruction mCBM labels
requested review from @a.bercuci
assigned to @se.gorbunov
added 9 commits
-
a791b51f...f1d209ec - 5 commits from branch
computing:master
- 3c802e82 - BUGFIX: input geo-file path improvement
- 11e421bd - mcbm2024 data recostruction chain:
- ed2e16e6 - mcbm2024 data recostruction chain:
- a660b4a1 - mcbm2024: a script for data reconstruction steps execution
Toggle commit list-
a791b51f...f1d209ec - 5 commits from branch
- Resolved by Alexandru Bercuci
Hi,
The idea with this "manual" was to have some specific info exposed to the user in the Git layout in a simple, human readable format. A bash script is also "human readable" but it also contains information which is not relevant (e.g. checks for your CbmRoot installation) - my impression is that I have to parse all the bash commands to see what are actually the steps. It is also written in a general format (switching for different run periods) which makes it a more general automatic script but a README instruction for this particular data (May mcbm24). Also, a script format is something which has to work, it means it has to be maintained. This is hard in general and the user frustration is big when things are crashing !
My conclusion is that the script which you introduced contains rather a "help menu" than a manual - list of instructions - for the user. I will try to write one along the lines we have discussed on Tuesday.
Thank you for removing the lines of the reco script. The rest are modifications in the CA part for which @se.gorbunov is more knowledgeable.
added 14 commits
-
a791b51f...b7ec10f8 - 10 commits from branch
computing:master
- 81df115b - BUGFIX: input geo-file path improvement
- 9cdc4664 - mcbm2024 data recostruction chain:
- 704c4a7f - mcbm2024 data recostruction chain:
- 1d9c082d - mcbm2024: a script for data reconstruction steps execution
Toggle commit list-
a791b51f...b7ec10f8 - 10 commits from branch
Dear @d.smith, @fweig, @f.uhlig, @p.-a.loizeau, @se.gorbunov, @s.zharko, @v.friese,
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