YAML configurable binary for digitization
This is a draft for a digitization binary as a follow up of the already converter offline reconstruction and at converter binaries. In contrast to the other two steps, I did not create the TaskFactory class, due to the very few lines to setup the digitization object, which is completely done within the Exec() function in Run.cxx.
After verification/cross-checking and finalizing run scripts (to be done after also transport step has been adapted to the new binary approach), this MR can replace the json configurable class, sim/response/base/CbmDigitizationConfig.h/.cxx as well as the corresponding macro macro/PWG/common/production/run_digi_json_config.C.
In comparison to the existing json config approach, two new options have been added: (1) storeAllTimeslices is actually used now (2) the time distribution of events can be set by user (old default was Poisson, now Poisson/Uniform can be chosen)
Final fixes have been applied and the outcome is identical to the currently used JSON approach.
Merge request reports
Activity
added Framework Simulation labels
requested review from @f.uhlig
assigned to @v.friese
added 206 commits
-
9a009840...4dfdfe09 - 192 commits from branch
computing:master
- 4dfdfe09...2955d02b - 4 earlier commits
- 431a2200 - Deactivation of one detector or all but one
- 1b31be83 - fix some features
- 7dc43e12 - remove unecessary stuff
- 3a742e69 - fix run
- 6b1c653a - std::vector in Config.h
- 9a82c86c - multiple input transport files possible
- d57a99b0 - update event mode
- 877be72d - clang format
- bd982bb2 - add noise option
- b944e018 - fix noise
Toggle commit list-
9a009840...4dfdfe09 - 192 commits from branch
added 1 commit
- de56c40a - proper converstion form TString to std::string
added 20 commits
-
de56c40a...8b306b63 - 3 commits from branch
computing:master
- 8b306b63...6da06568 - 7 earlier commits
- 355d8402 - fix run
- 514d6d9b - std::vector in Config.h
- ef28ec7b - multiple input transport files possible
- 3d868063 - update event mode
- 4e5435ee - clang format
- 7294ed87 - add noise option
- dd4e9cea - fix noise
- 70a65b7b - add external yaml-cpp to cmake
- 426c1580 - build fix
- cac5eeff - proper converstion form TString to std::string
Toggle commit list-
de56c40a...8b306b63 - 3 commits from branch
Dear @f.uhlig, @v.friese, @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.
added CodeOwners label
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz
- Resolved by Frederic Julian Linz