upgrade of TRD2D-FEE.

upgrade of TRD2D-FEE. Check MR @ cbmroot_geometry!174 (merged)

Dear @a.bercuci, @p.kaehler,

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

@p.-a.loizeau @d.emschermann

This parameters are needed for the mcbm fork to work on the Ni data from 23.05.22. Please consider porting them.

Alex

@a.bercuci @p.-a.loizeau If these parameter files are needed in the mCBM fork, shouldn't we better merge it there directly?

Unfortunately I don't know if the mcbm fork use other versions of parameters and geometry. What is clear though is that in the

setup_mcbm_beam_2022_05_23_nickel() the TRD tag is

// done TString trdGeoTag = "v22h_mcbm"; // TRD-2D May 2022 + 2x TRD-1D modules Also it is clear that this tag is only valid for mCBM as it describes a mcbm geometry !? Sorry, I can not help here.

@a.bercuci These are the 2 parameter repos used in the main repo an the mcbm fork:

main repo
origin	https://git.cbm.gsi.de/CbmSoft/cbmroot_parameter.git (fetch)

mcbm fork
origin	https://git.cbm.gsi.de/mcbm/cbmroot_parameter.git (fetch)

You can merge to mcbm if you pick a different branch in the MR GUI (mcbm/cbmroot_parameter).

I will rework the TRDs in the Nickel Setup this afternoon. The z-positions of the detectors need to be updated. We did measurements in the cave last week.

You can merge to mcbm if you pick a different branch in the MR GUI (mcbm/cbmroot_parameter).

done. check @

There is also a geometry update. Should I also push it to the mcbm fork (if it exists ?)

I will rework the TRDs in the Nickel Setup this afternoon. The z-positions of the detectors need to be updated. We did measurements in the cave last week.

Sure. This was in planning already for some time :)

@a.bercuci Please push geos to https://git.cbm.gsi.de/mcbm/cbmroot_geometry.git

Please push geos to https://git.cbm.gsi.de/mcbm/cbmroot_geometry.git

done check mcbm/cbmroot_geometry!28 (closed)

added 4 commits

• 92a2d4bf - [CI/CD] Add mCBM fork settings, probably to be removed before MR to upstream!
• b9c10f73 - Use Norberts TOF v21g for Uranium, refs #2483
• 341ea25e - Use TOF v21h derived from Norberts TOF v21g in setup 2022_08, refs #2516
• 5702ee40 - upgrade of TRD2D-FEE. Check MR @...

Compare with previous version

closed