-
Notifications
You must be signed in to change notification settings - Fork 180
MC: allow ITS3 to be simulated by dpg scripts #2128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
Signed-off-by: Felix Schlepper <felix.schlepper@cern.ch>
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
@f3sch: I will need some time to review this. Do you have an example where and how this is used? |
|
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-resourcesthis lets me produce full AO2Ds with ITS3 instead of ITS2. |
sawenzel
left a comment
There was a problem hiding this 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.
|
Thanks for the review. |
sawenzel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
In AliceO2Group#2128, I introduced a bug where the detectorList argument did not have a separating space after it.
In #2128, I introduced a bug where the detectorList argument did not have a separating space after it.
This adds the ability to specify the detectorList and also substitutes relevant parameters to easily setup ITS3 simulations