Draft: add benchmark for RICH IRT

close #81

• add RICH benchmark in benchmarks/rich
  • main script: run_irt.sh called by CI in process stage
    • runs scripts/simulate.py, an npsim wrapper specific for RICH tests
    • runs scripts/draw_histos.C, to check some raw results from scripts/simulate.py
    • runs gaudirun.py options/irt.py to run new Juggler algorithm IRTAlgorithm
    • eventually we should add some analysis script after gaudirun.py (start a new MR)
  • CI jobs:
    • 2 runs for each RICH, one for acceptance test and another for yield vs. momentum
    • artifacts from raw hits output to results/rich
    • more tests and artifacts to be added as we continue to develop IRT code
• new CI process jobs will fail until IRTAlgorithm is merged in Juggler
  • see pipeline https://eicweb.phy.anl.gov/EIC/benchmarks/reconstruction_benchmarks/-/pipelines/23625 where IRTAlgorithm step is disabled (artifacts results/rich)

assigned to @c.dilks

added 1 commit

• 3d3e69bf - re-enable rich benchmark

Compare with previous version

mentioned in merge request EIC/irt!1 (merged)

added 1 commit

• a31308f5 - add rich simulation script

Compare with previous version

added 1 commit

• 08c9b5cc - revert: benchmarks/rich/forward_hadrons.sh

Compare with previous version

added 1 commit

• d4c90b5c - generalize simulate.py for both RICHes

Compare with previous version

added 11 commits

• d4c90b5c...ff75cc6c - 6 commits from branch master
• 4648d0bc - move testIRT options file to benchmarks
• bc650d9b - re-enable rich benchmark
• 348cbc95 - add rich simulation script
• 5e1e1df6 - revert: benchmarks/rich/forward_hadrons.sh
• 7fa98047 - generalize simulate.py for both RICHes

Compare with previous version

mentioned in merge request EIC/irt!4 (closed)

added 1 commit

• d1afa468 - snake case

Compare with previous version

added 1 commit

• 7c1a9d30 - update options file, pulling changes from Alexander; needs testing

Compare with previous version

added 1 commit

• eeb253c1 - works up to IRT geometry import

Compare with previous version

added 1 commit

• 9548a04e - add control modes to run_irt.sh; update CI stage

Compare with previous version

added 1 commit

• 69f58a61 - add hits drawing script; update CI jobs

Compare with previous version

added 1 commit

• db318554 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• a1bd0829 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 705d3108 - debug npdet_info call

Compare with previous version

added 1 commit

• a19c83c5 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 8146604f - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• e10109a2 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 8dc1a30e - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• c36abb8d - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• e6714d72 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• b87e3861 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 38d575d4 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 8aec953f - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 75404e94 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• 50beffa5 - modified: benchmarks/rich/scripts/simulate.py

Compare with previous version

added 1 commit

• f7ef23aa - squash: debug npdet_info call

Compare with previous version

added 1 commit

• 151399bc - test draw_hits.C

Compare with previous version

added 1 commit

• fde26e21 - modified: config.yml

Compare with previous version

added 1 commit

• 4fb28d28 - modified: config.yml

Compare with previous version

added 1 commit

• df5f5b87 - add hits artifacts

Compare with previous version

added 1 commit

• 2ecf4688 - modified: benchmarks/rich/config.yml

Compare with previous version

added 1 commit

• cad444b6 - add momentum scan benchmarks

Compare with previous version

added 1 commit

• a9a912d8 - add momentum scan benchmarks

Compare with previous version

added 1 commit

• 402a2f61 - add raw histograms artifacts

Compare with previous version

added 1 commit

• e131c3c6 - cleanup artifacts

Compare with previous version

added 1 commit

• a29831a9 - cleanup

Compare with previous version

added priority label

added 1 commit

• 71e75e63 - disable IRT geometry root config file

Compare with previous version

added 1 commit

• e5462393 - add name

Compare with previous version

added 1 commit

• 5b7ab6ab - generalize options file for both RICHes

Compare with previous version

added 1 commit

• 8d61968b - sync options file with irt version

Compare with previous version

added 1 commit

• 054cb65b - add param print

Compare with previous version

added 20 commits

• 193b5399 - 1 commit from branch master
• 04722cbd - move testIRT options file to benchmarks
• 1a829e31 - re-enable rich benchmark
• 19324787 - add rich simulation script
• f2ccdbaf - revert: benchmarks/rich/forward_hadrons.sh
• a77bb9ad - generalize simulate.py for both RICHes
• 9bfdc720 - snake case
• 8169a401 - update options file, pulling changes from Alexander; needs testing
• b0229e4a - works up to IRT geometry import
• b4771e59 - add control modes to run_irt.sh; update CI stage
• 15a44d4c - add hits drawing script; update CI jobs
• 44d853c9 - squash: debug npdet_info call
• 370572b4 - add raw histograms artifacts
• ba119be7 - cleanup artifacts
• 14b39d68 - cleanup
• 51a6e9e5 - disable IRT geometry root config file
• 6385fa1e - add name
• 69caedc5 - generalize options file for both RICHes
• f5f2f295 - sync options file with irt version
• 368c2278 - add param print

Compare with previous version

added 1 commit

• a6142e31 - disable IRTAlgorithm

Compare with previous version

added 1 commit

• 9aff7b63 - re-enable IRTAlgorithm

Compare with previous version

added 1 commit

• 0056b530 - revert: .gitignore

Compare with previous version

marked this merge request as ready

changed the description

approved this merge request

@wdconinc we are marking this MR as draft again, since @chchatte plans to add more benchmark plots, e.g. yield vs. eta

marked this merge request as draft

added 1 commit

• 9203d91a - adjust hit drawing

Compare with previous version

added 28 commits

• 9203d91a...8883eac0 - 5 commits from branch master
• 30a25827 - move testIRT options file to benchmarks
• 6543cba0 - re-enable rich benchmark
• 495b1c1d - add rich simulation script
• 5ed580c7 - revert: benchmarks/rich/forward_hadrons.sh
• e19b1d75 - generalize simulate.py for both RICHes
• 0553e4fa - snake case
• 2918060f - update options file, pulling changes from Alexander; needs testing
• 150b099b - works up to IRT geometry import
• d417548b - add control modes to run_irt.sh; update CI stage
• 1bb68f06 - add hits drawing script; update CI jobs
• 121af7d4 - squash: debug npdet_info call
• 00d30de7 - add raw histograms artifacts
• 9a194883 - cleanup artifacts
• 252fc609 - cleanup
• 4b3aad49 - disable IRT geometry root config file
• 8c6d5691 - add name
• 5115858b - generalize options file for both RICHes
• 7eb16ebd - sync options file with irt version
• 04380c4f - add param print
• 0a584027 - disable IRTAlgorithm
• 91a57587 - re-enable IRTAlgorithm
• 425d5d3d - revert: .gitignore
• 7169478a - adjust hit drawing

Compare with previous version

removed priority label

added 1 commit

• adc96d5b - remove xenv call from run_irt.sh

Compare with previous version

added 1 commit

• 8ee170d0 - eRICH -> pfRICH

Compare with previous version

added 1 commit

• 55daf0ad - test npsim and draw_histos.C

Compare with previous version

added 1 commit

• a9f4773c - add library loaders to draw_histos.C

Compare with previous version

added 1 commit

• b5ae950e - re-enable juggler in CI

Compare with previous version

added 1 commit

• 7afe261c - copy options files from irt repo

Compare with previous version

added 1 commit

• c3c18127 - add env vars to options files; decrease verbosity

Compare with previous version

added 1 commit

• 574d1f4e - improve exit handling of run_irt.sh

Compare with previous version

added 1 commit

• 6641aba7 - temporarily disable juggler

Compare with previous version

added 1 commit

• 3612c2b9 - re-enable juggler

Compare with previous version

added 1 commit

• 020c4137 - change PhotoMultiplierHit to TrackerHit

Compare with previous version