Skip to content
Snippets Groups Projects

Fix bug in configuration for plume lumi algorithm

Merged Daniel Charles Craik requested to merge dcraik-lumi-plume-bugfix into master
All threads resolved!

Adds two lines to the configuration of the plume lumi counter algorithm, which were accidentally omitted in the original version. Also adds one omitted line to MakeLumiSummary.cu, which allows the plume counters to be written to the output bank.

Additionally fixed a bug where the plume overthreshold bits counters would overflow.

Plume lumi counters produced successfully in offline and online tests.

Edited by Daniel Charles Craik

Merge request reports

Merged by Rosen MatevRosen Matev 2 years ago (Apr 19, 2023 7:56am UTC)

Merge details

  • Changes merged into master with 3a3f797f.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading