Skip to content

Honor detector inclusion list for FT0,FV0,EMC,CTP#2306

Merged
sawenzel merged 1 commit intoAliceO2Group:masterfrom
mpuccio:master
Mar 24, 2026
Merged

Honor detector inclusion list for FT0,FV0,EMC,CTP#2306
sawenzel merged 1 commit intoAliceO2Group:masterfrom
mpuccio:master

Conversation

@mpuccio
Copy link
Contributor

@mpuccio mpuccio commented Mar 23, 2026

No description provided.

@github-actions
Copy link

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
Copy link
Contributor

@mpuccio : Could you please provide a version of the PR without whitespace changes? We should stick to minimal commits so as to avoid merge-conflicts etc.

@sawenzel
Copy link
Contributor

Also, could you please give an example of a MC workflow where the treatment was previously wrong?

Copy link
Contributor

@sawenzel sawenzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see other comments

@mpuccio
Copy link
Contributor Author

mpuccio commented Mar 24, 2026

Hi @sawenzel, my problem was when running something like:

        ${O2DPG_ROOT}/MC/bin/o2dpg_sim_workflow.py -eCM 13600  -seed 42 -col pp -gen pythia8 -proc inel -tf ${NTFS} \
                                                               -ns ${NEVENTS} -e ${SIMENGINE} --condition-not-after 3385078236000 --timestamp 1757503805356 -run 566030 \
                                                               -j ${NWORKERS} -interactionRate ${INTRATE} -confKey "$CONFKEY" \
                                                               --skipModules ZDC PHS CPV MID MCH EMC \
                                                                                 --readoutDets ITS,CTP,TPC,TRD,TOF,FT0,MFT,FV0,FDD \
                                                               --no-combine-smaller-digi --overwrite-config customize.json -bcPatternFile bcPattern.root

EMC was not skipped, hence the modification.

@mpuccio mpuccio requested a review from sawenzel March 24, 2026 08:50
@sawenzel sawenzel merged commit 8d9bcc2 into AliceO2Group:master Mar 24, 2026
7 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.

2 participants