Skip to content
Snippets Groups Projects

TRD2D geometries for CFV (day1) version (23b) and for full setup (24a)

Open Alexandru Bercuci requested to merge a.bercuci/cbmroot_geometry:trd2d-full into master

Please insert x between square brackets to tick the boxes, add a meaningful description if the title is not enough and check the outcome with the "Preview" tab

In case you committed a new geometry:

  • Ran the scripts for materials and radiation length check on my new geometry file? (and read carefully the output of course)
    bash ci_scripts/check_radlen.sh <path>/<file>.geo.root
  • Added also the corresponding creation macro?
  • Checked that the geometry does not contains unexpected overlaps, e.g. either at end of the creation macro or with the something similar to the mCBM macro
    macro/mcbm/mcbm_check_overlaps.C

In case you committed a new setup file:

  • Checked that all geometries listed are present in the repository?
  • Ran the CBMROOT macro/geometry/check_media.C on the corresponding .geo.root file? (output by the transport macro)

Depends on the following:

Edited by Alexandru Bercuci

Merge request reports

Members who can merge are allowed to add commits.

Merge request pipeline #31995 passed

Merge request pipeline passed for 4e83e6c1

Ready to merge by members who can write to the target branch.
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading