Skip to content

[PWGJE,EMCAL-670] Add BC selection to EMCal QA task#10847

Merged
fjonasALICE merged 4 commits intoAliceO2Group:masterfrom
mhemmer-cern:EMCAL-670
Apr 16, 2025
Merged

[PWGJE,EMCAL-670] Add BC selection to EMCal QA task#10847
fjonasALICE merged 4 commits intoAliceO2Group:masterfrom
mhemmer-cern:EMCAL-670

Conversation

@mhemmer-cern
Copy link
Collaborator

  • Add Core/utilsBcSelEMC.h where EMCEventSelection struct is defined with configurables to select BCs based on different selection bits.
  • Add EMCEventSelection to emcEventSelectionQA.cxx

- Add Core/utilsBcSelEMC.h where EMCEventSelection struct is defined with configurables to select BCs based on different selection bits.
- Add EMCEventSelection to emcEventSelectionQA.cxx
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 38f9bb1 at 2025-04-14 20:07:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10847-slc9_x86-64/0/PWGJE/Core/utilsBcSelEMC.h:152:33: error: unused parameter 'bc' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

- Remove functions that were not needed
- Add histograms to check if EMCal was readout but BC was not selected and if EMCal had cells, but BC was not selected
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 3a251fb at 2025-04-15 12:08:

## sw/BUILD/arrow-latest/log
/sw/BUILD/52f6992da5d7a8f7fc221f3ae24c7c47ec53d57a/arrow/src_tmp/cpp/src/gandiva/precompiled/arithmetic_ops.cc:18:10: fatal error: 'cmath' file not found
ninja: build stopped: subcommand failed.

Full log here.

- Rename Trigger to Sel8 for the hRejctio histogram
@fjonasALICE fjonasALICE self-requested a review April 16, 2025 13:18
@fjonasALICE fjonasALICE merged commit 5179f01 into AliceO2Group:master Apr 16, 2025
14 checks passed
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request May 17, 2025
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants