Skip to content

Conversation

@f3sch
Copy link
Contributor

@f3sch f3sch commented Oct 2, 2025

In #2128, I introduced a bug where the detectorList argument did not have a separating space after it.
This leads to an incorrect construction of the cmd string --detectorList ALICE2--configKeyValues for the qed background sim. I only tested my changes in pp not in Pb-Pb...sorry

In AliceO2Group#2128, I introduced a bug where the detectorList argument did not have a separating space after it.
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

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

@f3sch f3sch changed the title fix detectorList argument for PbPb fix detectorList argument for Pb-Pb Oct 2, 2025
@sawenzel
Copy link
Contributor

sawenzel commented Oct 2, 2025

No worries. We should improve the CI and test various different workflows.

@sawenzel sawenzel merged commit 224078c into AliceO2Group:master Oct 2, 2025
6 checks passed
@f3sch f3sch deleted the patch-1 branch October 2, 2025 11:03
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.

2 participants