Skip to content

[PWGEM] Add configurable for min. number of photons in EOI - #17698

Merged
jokonig merged 1 commit into
AliceO2Group:masterfrom
jokonig:master
Sep 1, 2026
Merged

[PWGEM] Add configurable for min. number of photons in EOI#17698
jokonig merged 1 commit into
AliceO2Group:masterfrom
jokonig:master

Conversation

@jokonig

@jokonig jokonig commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator
  • This is to reduce the datasize of derived data. Use min. number of photons >= 2 for pi0 etc. analysis

- This is to reduce the datasize of derived data. Use min. number of photons >= 2 for pi0 etc. analysis
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 3 errors, ⚠️ 20 warnings, 🔕 0 disabled

@jokonig
jokonig enabled auto-merge (squash) September 1, 2026 13:40

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto-approving on behalf of @jokonig.

@jokonig
jokonig merged commit b7efad9 into AliceO2Group:master Sep 1, 2026
13 of 15 checks passed
@alibuild

alibuild commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 2ee288d at 2026-09-01 17:28:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 2 errors and 0 warnings.'
Found 2 errors and 0 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/81257a878131515f90b1bf4bb58184758d96ee7c/O2Physics-code-check/errors.txt
PWGEM/Dilepton/TableProducer/filterEoI.cxx:22:1: error: included header ASoAHelpers.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGEM/Dilepton/TableProducer/filterEoI.cxx:32:1: error: included header Vector4D.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 2 -gt 0 ]]
++ exit 1
--

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants