[RTADPA BW Tests] Introducing an Hlt1-bandwidth test via Moore_in_Allen
Adds a new BW test: for Nominal Hlt1 via Moore_in_Allen.
Makes several changes to make the scripts less 'X if Hlt2 else Y'.
Changes the main-page, so that the links at the bottom are dependent upon process. (This removes the dead-link on sprucing page, and prevents confusion on the new hlt1 page)
cc: @rjhunter
Goes with: lhcb/Moore!2528 (merged), lhcb-core/LHCbNightlyConf!1094 (merged) and lhcb-core/LHCbPR2HD!263 (merged)
ToDo:
-
Insert process-dependence on N_EVTS for now. see: #12 (closed) -
CI-test -
BW chat for review (@shunan) -
Post-Merge -
Allen label -
Dmitry Popov "magic step" to register new test in LHCbPR
-
Merge request reports
Activity
assigned to @lugrazet
mentioned in merge request lhcb/Moore!2528 (merged)
added 50 commits
-
a02c305f...dfd6b02c - 42 commits from branch
master
- 88edfefd - option for moore_bw_test hlt1
- ca09fc61 - Moore_hlt1_bw.sh
- 9f1f9a94 - Correct extra_opts for Hlt1
- 54451341 - Functioning Hlt1 Job :)
- f800b1b7 - hlt1-line-descriptives-hack
- 533d241c - struggling to define streams
- 081a34cd - Successful step 4 + some argparse changes
- e61c3d83 - Successful part6. Need to understand bw_test_page -s
Toggle commit list-
a02c305f...dfd6b02c - 42 commits from branch
- Resolved by Luke Grazette
@rjhunter This is working remotely now
.From my perspective there's several things I'd like your thoughts on before we test it and progress to merging.
Input file. Right now as a stand-in I chose just the default hlt1 input file I could see in the Moore tests, not sure what we should actually use.
To download the files locally, there's a function
Mooreconf.qmtest.context:download_mdf_inputs_locally
which is used.
This function isn't actually that related to the object being anmdf
file, so I added a few changes to allow downloadingdigi
through the function as well.
It occurred to me that the least intrusive method would have been to add a new functiondownload_digi_inputs_locally
but that seemed too duplicative for not enough reason. Personally my only issue with the curent 'hack' is that the namedownload_mdf_inputs_locally
is now slightly incorrect but I'd rather not have to cause endless changes across projects by changing the name to something that would now be accurate. I wonder if you agree with me and the change made?Edited by Luke Grazette