Skip to content

[EMCAL-1054] Enable BC task in simulations#1359

Merged
chiarazampolli merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-1054
Dec 4, 2023
Merged

[EMCAL-1054] Enable BC task in simulations#1359
chiarazampolli merged 1 commit into
AliceO2Group:masterfrom
mfasDa:EMCAL-1054

Conversation

@mfasDa
Copy link
Copy Markdown
Contributor

@mfasDa mfasDa commented Dec 1, 2023

Include EMCAL BC QC in simulations in case
CTP is included.

@mfasDa
Copy link
Copy Markdown
Contributor Author

mfasDa commented Dec 1, 2023

Needs AliceO2Group/AliceO2#12364, therefore handled as separate PR.

@mfasDa
Copy link
Copy Markdown
Contributor Author

mfasDa commented Dec 1, 2023

Results from local tests shown in #1352

Include EMCAL BC QC in simulations in case
CTP is included.
@mfasDa
Copy link
Copy Markdown
Contributor Author

mfasDa commented Dec 1, 2023

Needs AliceO2Group/QualityControl#2060 in order to provide different trigger class names for min. bias triggers, or to allow an or of 2 min. bias trigger classes at the same time.

Reasons:

  • Trigger class names were changing in time, we need to be able to allow different classes and match the with the corresponding trigger configuration of that run
  • In pp we used 2 trigger classes as min. bias trigger (C0TVXTSC,C0TVXTCE), we need to be able to accept triggers if either of them fires.

@mfasDa
Copy link
Copy Markdown
Contributor Author

mfasDa commented Dec 4, 2023

The PR in QC is tagged as v1.26.1, the PR can be merged now.

@chiarazampolli
Copy link
Copy Markdown
Collaborator

Hello @benedikt-voelkel , @noferini ,
The CI is full of

8-Background generator seed:  825338745
9-Timeframe 1 seed:  825338746
10-o2dpg_sim_workflow: Error! Interaction rate not >0 !!!
11:[FATAL]: O2DPG_TEST Workflow creation failed

I am merging this one now.
Chiara

@chiarazampolli chiarazampolli merged commit d23b6cc into AliceO2Group:master Dec 4, 2023
@mfasDa mfasDa deleted the EMCAL-1054 branch December 4, 2023 09:21
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