TOF v24a requires a new line in the CbmTofGeoHandler. Added new setup to macro/mcbm/mcbm_transport.C
Merge request reports
Activity
requested review from @p.-a.loizeau
assigned to @e.clerkin
assigned to @p.-a.loizeau and unassigned @e.clerkin
requested review from @e.clerkin and removed review request for @p.-a.loizeau
Should probably be merged after we add a commit with the hash bump from CbmSoft/cbmroot_geometry!253 (merged)
- Resolved by Pierre-Alain Loizeau
In fact, @d.emschermann, could you please also add it to the list of setups tested in
macro/mcbm/CMakeLists.txt
? this way we get the full CI on it
added 1 commit
- Resolved by Pierre-Alain Loizeau
You will need to also add an entry for the bmon of the new geometry in the ignore list of the overlaps checkers.
You can do this by adding the following at l.161 ofmacro/geometry/check_overlaps.C
:cave/pipe_v19f_0/vacu20_1 overlapping cave/tof_v24a_mcbm_0/tof_v24a_mcbmStand_1/module_5_0
added 1 commit
enabled an automatic merge when the pipeline for c3677a58 succeeds
Dear @n.herrmann, @i.deppner, @f.uhlig, @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.