Skip to content
Snippets Groups Projects

QEE: Lower Hlt2DiMuonNoIP lines rates by filtering on Hlt1DiMuonNoIP

Merged Adrian Casais Vidal requested to merge acasaisv/fix-dpmumu-rate into 2024-patches

Adds Hlt1 filter to Hlt2DiMuonNoIP lines and lowers the rate significantly.

Old BW estimate over minbias:

Turbo BW: 119.333 kHz; 535.642 MB/s
Full BW: 1.262 kHz; 317.453 MB/s

New BW estimate over minbias:

Turbo BW: 29.298 kHz; 159.798 MB/s
Full BW: 1.042 kHz; 312.250 MB/s

I changed the full rescale to allow it to capture more events keeping a similar BW footprint. If this is a problem we can also tight the FULL stream.

Edited by Adrian Casais Vidal

Merge request reports

Loading
Loading

Activity

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

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Luke Grazette mentioned in merge request !3354 (merged)

    mentioned in merge request !3354 (merged)

  • Unfortunately this isn't ready in time. I would suggest please continuing to get this ready (Fix the CI-pipeline, report the BW anew with the current changes) and we can get this merged as soon as possible after the deadline.

  • added 1 commit

    Compare with previous version

  • I'm so sorry about the pipeline. It should be fixed now.

    • Resolved by Ross John Hunter

      Hi @acasaisv, as @lugrazet said above, unfortunately we can't get this into qee_upgrade_24 in time for tomorrow's deadline (our QEE MR is already tested and ready to be merged).

      Is this ready for merge from your perspective? I think if so then we should just point this at 2024-patches and we see if the powers that be will let us merge it given the potentially massive saving.

      Edited by Ross John Hunter
  • added RTA label

  • Adrian Casais Vidal changed the description

    changed the description

  • Adrian Casais Vidal changed target branch from qee_upgrade_24 to 2024-patches

    changed target branch from qee_upgrade_24 to 2024-patches

  • requested review from @lugrazet, @rjhunter, and @dzuliani

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