Draft: Fix generation of RICH geometry file

Use the correct TGeo unit system. Use TVolume->Export to store the RICH volume in the ROOT file. Add the correct placement matrix according to the shift of the coordinate system. Also create the file needed for the media check. Create all needed materials prior to the import from the gdml file such that they are properly exported to the ROOT file

changed milestone to %DEC21

added BugFix Geometry labels

marked this merge request as draft

@s.lebedev,

please review the changes. This is the first part of the changes discussed in #2425

added 1 commit

• 9cfab8a3 - Fix generation of RICH geometry file

Compare with previous version

@f.uhlig

I tested this new macro and it did not work for me:

1. I copied rich_v17a_1e.gdml and changed names of top volume to rich_v21a
2. Then I run your new macro, and I got an error in the end

/Users/slebedev/Development/cbm/git/cbmroot/macro/geometry/create_medialist.C:57:21: error: use of undeclared identifier 'RemoveGeoManager'
  if (removegeomgr) RemoveGeoManager();

But this error is not a problem.

3. When running the simulation with a newly produced geometry file I have too many points in the RICH and no points in STS.

I expect that the is still problem with materials.

To solve 2) you need to have the .rootrc from macro/input. Either copy or link the file. But as you said this isn't a problem.

Concerning 3) I have a different result. I have STSPoints as well as RICHPoints in the transport file. I will add the plots to issue #2425.

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

I close this MR. The MR is superseded by !715 (merged) which adds the generated geometry as well as the file needed for the media check.

closed

mentioned in merge request !715 (merged)