Skip to content

Conversation

@f3sch
Copy link
Contributor

@f3sch f3sch commented Sep 24, 2025

This adds the ability to specify the detectorList and also substitutes relevant parameters to easily setup ITS3 simulations

Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
@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

@f3sch: I will need some time to review this. Do you have an example where and how this is used?

@f3sch
Copy link
Contributor Author

f3sch commented Sep 24, 2025

Hi, I am running it locally like this:

${O2DPG_ROOT}/MC/bin/o2dpg_sim_workflow.py -eCM 13600 -seed 42 -col pp -gen pythia8 -proc inel -tf ${NTFS} \
    -ns ${NEVENTS} -e ${SIMENGINE} -run 303901 \
    -j ${NWORKERS} -interactionRate ${INTRATE} \
    --first-orbit 0 --orbits-early 0 --no-combine-smaller-digi -confKey "$CONFKEY" \
    --detectorList ALICE2.1 --anchor-config config_303901.json
${O2DPG_ROOT}/MC/bin/o2_dpg_workflow_runner.py -f workflow.json -tt aod ${MEMLIMIT} ${CPULIMIT} --optimistic-resources

this lets me produce full AO2Ds with ITS3 instead of ITS2.
Since none of the ccdb objects are currently correct for the ITS3 run ranges, I set up a repository to collect the ITS3 configurations and ccdb objects.

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.

Looks good and indeed will enable Run4 ITS with almost the same logic. Will approve once the minor points are clarified.

@f3sch
Copy link
Contributor Author

f3sch commented Oct 2, 2025

Thanks for the review.
I verified that everything up&including to the AOD creation is correct.
Points that have to be clarified later is that ITS3 cannot run the QC task yet (but ok this just has to be adapted when we are in Run 4).

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.

LGTM

@sawenzel sawenzel merged commit d0e1981 into AliceO2Group:master Oct 2, 2025
7 checks passed
@f3sch f3sch deleted the mc/its3 branch October 2, 2025 07:13
f3sch added a commit to f3sch/O2DPG that referenced this pull request Oct 2, 2025
In AliceO2Group#2128, I introduced a bug where the detectorList argument did not have a separating space after it.
sawenzel pushed a commit that referenced this pull request Oct 2, 2025
In #2128, I introduced a bug where the detectorList argument did not have a separating space after it.
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