add ICD correction to hitpriducer of RICH and add functionality to Macro.
Merge request reports
Activity
Dear @s.lebedev,
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
assigned to @p.-a.loizeau
added 26 commits
-
daf0a684...5acfd390 - 25 commits from branch
computing:master
- f43ed2cb - add ICD correction to hitpriducer of RICH and add functionality to Macro.
-
daf0a684...5acfd390 - 25 commits from branch
could you please have a look. From my side the MR looks okay.
added mCBM label
added Reconstruction label
- Resolved by Pierre-Alain Loizeau
Looks good to me. Not a RICH expert, but it seems simple enough to be ok.
@a.weber Is it something that should be used for the latest mCBM data?
I yes, I think it would be better to merge it to the mCBM fork instead of the main master.
added 5 commits
-
f43ed2cb...c3875e0b - 4 commits from branch
computing:master
- 54a82dc7 - add ICD correction to hitpriducer of RICH and add functionality to Macro.
-
f43ed2cb...c3875e0b - 4 commits from branch
I would propose to merge it here. If needed also for mCBM before rebasing from the master it should be save to merge the same MR also to the mCBM fork. Git is able to resolve such a situation without problems.
added 15 commits
-
54a82dc7...bf6a2a95 - 14 commits from branch
computing:master
- 3d559ad4 - add ICD correction to hitpriducer of RICH and add functionality to Macro.
-
54a82dc7...bf6a2a95 - 14 commits from branch
mentioned in merge request mcbm/cbmroot!51 (merged)