Skip to content
Snippets Groups Projects

Add EventView handling to AvalancheScheduler

Merged Benjamin Michael Wynne requested to merge bwynne/Gaudi_gaudi:ScheduleEventViews into master
All threads resolved!

Mostly discussed here: https://its.cern.ch/jira/browse/GAUDI-1306

Recent design presentation https://indico.cern.ch/event/632442/contributions/2718811/attachments/1526541/2387104/EventViews.pdf

Older, introductory material https://indico.cern.ch/event/578275/contributions/2342808/attachments/1416698/2169616/EventViews.pdf

Adds a method to IScheduler.h to attach EventViews to the PrecedenceRulesGraph, so that parts of the graph can be processed multiple times per event (or skipped entirely).

No measurable performance impact for non-view workloads.

May require resolution of other MRs such as !449 (closed)

Edited by Benjamin Michael Wynne

Merge request reports

Pipeline #247101 passed

Pipeline passed for 1b161ad8 on bwynne:ScheduleEventViews

Merged by avatar (Jun 18, 2025 1:08pm UTC)

Loading

Pipeline #254532 passed

Pipeline passed for 097a8a29 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
  • Marco Clemencic changed milestone to %v30r1

    changed milestone to %v30r1

  • added 39 commits

    • 53f9db66...a1f1375a - 36 commits from branch gaudi:master
    • 80ad7f7e - Merge remote-tracking branch 'upstream/master' into ScheduleEventViews
    • b918b94d - Merge remote-tracking branch 'upstream/master' into ScheduleEventViews
    • 1d7cb548 - Re-arranged the code to avoid needing slotAscend() and slotDescend().…

    Compare with previous version

  • added 15 commits

    • 1d7cb548...339e7bf8 - 14 commits from branch gaudi:master
    • bfc2b028 - Merge remote-tracking branch 'upstream/master' into ScheduleEventViews

    Compare with previous version

  • Illya Shapoval resolved all discussions

    resolved all discussions

  • Benjamin Michael Wynne resolved all discussions

    resolved all discussions

  • added 1 commit

    • 77564801 - Pointers initialised with nullptr

    Compare with previous version

  • Edited by Software for LHCb
  • added 1 commit

    • 1b161ad8 - I forgot to re-include protection for DecisionNodes within views

    Compare with previous version

  • Marco Clemencic approved this merge request

    approved this merge request

  • Marco Clemencic mentioned in commit 097a8a29

    mentioned in commit 097a8a29

  • Illya Shapoval mentioned in merge request !539 (merged)

    mentioned in merge request !539 (merged)

  • Please register or sign in to reply
    Loading