Skip to content

Draft: RD developments towards August 2024 MD

Miguel Ramos Pernas requested to merge rd-devel into 2024-patches

Depends on lhcb-datapkg/ParamFiles!100

Overall impact on bandwidth

To be filled once the bandwidth and throughput tests have run.

Goals

Include modifications to be included for the period after the MD happening in August. The internal deadline is set to the 17th of July. Follows !3394 (merged). These changes include tweaks to lines, bug fixes and the inclusion of downstream and upstream reconstruction in different (new) lines.

Contributing

If you want to contribute to this MR, please speak first to @mramospe and @matzeni and explain your goals. We should avoid having clashes across different files, so if your line is defined in a module where other lines live, please coordinate among the contributors to the file to avoid conflicts. Once you are given green light to do changes, please do so and include in the description of this MR a little summary indicating the file(s) that you have modified as well as the changes to the lines. If you are changing a lot of different lines, please consider making a MR targeting the rd-devel branch. Note that we will not be able to run the central tests on that MR, but it will facilitate the review and the testing locally.

Any contributions to this MR should be ready by the 17th of July, in such a way that the impact on the rate, bandwidth and throughput has been studied and is within the thresholds.

All the additions need to be tested locally, including:

  • Impact on the bandwidth and rate: if including new lines, these must be well justified (i.e. the physics case must not be already covered by other existing lines) and have rates and bandwidths below 100 Hz and 10 MB/s, respectively. This must be tested on real data. Follow the instructions at the RD migration meeting for advice on how to proceed. If you are tweaking existing lines the same thresholds apply.
  • Impact on the throughput: comparing the time with the execution of 2024-patches with no extra modifications.
  • (optional) Determine efficiencies: if there is simulation already available, it will be interesting to know the efficiencies so we ensure that there is no bug in the selections.

Do not rebase or force-push to this branch without the consent of @mramospe and/or @matzeni, since this might lead to losses on the changes

Please follow the same format as in !3394 (merged).

Changes

Global

Nothing at the moment.

Edited by Miguel Ramos Pernas

Merge request reports