Skip to content
Snippets Groups Projects

[RTADPA BW Tests] New Sprucing bandwidth test using latest-available HLT2 input.

Merged Luke Grazette requested to merge lugrazet-BW-spruce-ydayshlt2 into master

cc: @rjhunter.

related to: lhcb/Moore!2763 (merged) + lhcb-core/LHCbPR2HD!270 (merged) + lhcb-core/LHCbNightlyConf!1121 (merged)

Changes:

Adds a new sprucing test, Moore_spruce_latest_bandwidth, which uses the Hlt2 nominal full-stream output which was uploaded to eos on the previous day.

Added a new script, generate_hlt2_fullstream_metadata.py, which runs only for Hlt2 nominal tests and writes the metadata.yaml based on information from the full-stream output of the test currently running. This metadata.yaml is used as an input config for the new sprucing test.

  • a little safety around division in zero from the calculation of overlap_matrix, never comes up in the actual tests because every stream has a non-zero retention in 100k events, but when running test jobs it can come up and it was annoying me.

  • changes to facilitate the new test + metadata.

ToDo:

  • Maybe revisit the N_EVTS and calculate via a gaudipython job?
    • Wait to see if BWTesters know a way to tell the N_EVTS available in a file.
    • Else, adapt list_event_numbers to extract the N_EVTS by iterating a gaudipython job.
  • Rebase after !362 (merged) was merged

Closes #12 (closed)

Edited by Luke Grazette

Merge request reports

Pipeline #6639844 passed

Pipeline passed for 9e60b1c8 on lugrazet-BW-spruce-ydayshlt2

Approved by

Merged by Eduardo RodriguesEduardo Rodrigues 1 year ago (Dec 20, 2023 10:32am UTC)

Merge details

  • Changes merged into master with 627d1b28 (commits were squashed).
  • Deleted the source branch.

Pipeline #6643900 passed

Pipeline passed for 627d1b28 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Luke Grazette added 1 commit

    added 1 commit

    • d3be07ce - some refactorisation of strings/names in/from bandwidth_helpers

    Compare with previous version

  • Luke Grazette changed the description

    changed the description

  • Luke Grazette added 3 commits

    added 3 commits

    Compare with previous version

  • Luke Grazette changed the description

    changed the description

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading