Skip to content

Give the QED interaction spec the size of the QED event pool - #2440

Merged
sawenzel merged 1 commit into
AliceO2Group:masterfrom
sawenzel:qed-roundrobin-event-ids
Aug 24, 2026
Merged

Give the QED interaction spec the size of the QED event pool#2440
sawenzel merged 1 commit into
AliceO2Group:masterfrom
sawenzel:qed-roundrobin-event-ids

Conversation

@sawenzel

Copy link
Copy Markdown
Contributor

This fixes a problem in the QED specification handed to the collision context tool.

  • The MC number string was 10000000:NEventsQED, so the round robin wrapped the QED event IDs at 10000000 instead of at the number of events the workflow simulates, and the MC labels named QED events which do not exist.
  • Both numbers are NEventsQED now, which also repairs the labels on the O2 versions already deployed.

https://its.cern.ch/jira/browse/O2-7132

This fixes a problem in the QED specification handed to the collision context
tool.

- The MC number string was 10000000:NEventsQED, so the round robin wrapped the
  QED event IDs at 10000000 instead of at the number of events the workflow
  simulates, and the MC labels named QED events which do not exist.
- Both numbers are NEventsQED now, which also repairs the labels on the O2
  versions already deployed.

https://its.cern.ch/jira/browse/O2-7132
@sawenzel
sawenzel requested a review from jackal1-66 as a code owner August 24, 2026 11:36
@github-actions

Copy link
Copy Markdown

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@sawenzel
sawenzel merged commit c2a97f3 into AliceO2Group:master Aug 24, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant