Skip to content
Snippets Groups Projects

Add possibility to create mCBM setup geo root file from only the runID + some cleanup

Merged Pierre-Alain Loizeau requested to merge p.-a.loizeau/cbmroot:mcbm_run2setup into master

This is groundwork for @a.toia changes on adding an mCBM reco step to the CI/CD chain. I would have also put her as reviewer if not restricted by the Gitlab roles.
More macros were changed as this was also a good opportunity to remove some code duplication in non compiled code.

  • Add utils function for RunId to setup mapping
    • Add new file and namespace in utils part of the CbmBase library
    • Add Mcbm::GetSetupFromRunId function
    • Add Mcbm::Dummy class to force loading of the library n macros and circumvent deprecation of functions dict in Root6/Cling
    • Update all macros with the if/else mapping to use the new function and catch exception for unmapped runs
  • Add macro to create the setup geo root files based on RunId
    • For now making call to mcbm_transport.C from a different folder (hardcoded path) + running 1 event
    • Call should be replaced on the long term with an INIT only method (no need to run a simulation to get a geometry)

Performed tests:

  • Proper mapping in macro/run/run_unpack_tsa.C for run 2391 (on run4)
  • Failed mapping and exception properly caught in macro/run/run_unpack_tsa.C for run 900
  • Failed mapping and exception properly caught in macro/run/run_unpack_tsa.C for run 2500
  • Proper generation of the geo.root files with the expected names and path for run 2391
  • No regeneration if geometry file already there
  • CI test with run 2391
Edited by Pierre-Alain Loizeau

Merge request reports

Loading
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