Skip to content

[PWGCF] Remove extra event selection condition - #17994

Merged
abmodak merged 3 commits into
AliceO2Group:masterfrom
abmodak:remove-evtsel
Sep 21, 2026
Merged

abmodak merged 3 commits into
AliceO2Group:masterfrom
abmodak:remove-evtsel

Conversation

@abmodak

@abmodak abmodak commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Removed the reject flange event check from FT0 loop.
@github-actions github-actions Bot added the pwgcf label Sep 21, 2026
@github-actions github-actions Bot changed the title Remove extra event selection condition [PWGCF] Remove extra event selection condition Sep 21, 2026
@github-actions

Copy link
Copy Markdown

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

Please consider the following formatting changes to AliceO2Group#17994
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 8e238ae at 2026-09-21 17:04:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 55 warnings.'
Found 1 errors and 55 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/4a537ca297b497717bdf7aa6d0891af424a961a5/O2Physics-code-check/errors.txt
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:65:1: error: included header bitset is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 55 -gt 0 ]]
++ cat /sw/BUILD/4a537ca297b497717bdf7aa6d0891af424a961a5/O2Physics-code-check/warnings.txt
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:96:10: warning: variable 'axisEvent' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:97:10: warning: variable 'axisTrackSel' is non-const and globally accessible, consider making it const [cppcoreguidelines-avoid-non-const-global-variables]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:105:8: warning: constructor does not initialize these fields: ccdb, pdg [cppcoreguidelines-pro-type-member-init]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:265:30: warning: initializer for member 'cfgSgCuts' is redundant [readability-redundant-member-init]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:358:41: warning: narrowing conversion from 'float' to 'int' [bugprone-narrowing-conversions]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:360:38: warning: narrowing conversion from 'float' to 'int' [bugprone-narrowing-conversions]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:406:35: warning: initializer for member 'ft0gainvalues' is redundant [readability-redundant-member-init]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:446:34: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:448:44: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:554:51: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:650:11: warning: implicit conversion 'float' -> 'bool' [readability-implicit-bool-conversion]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:650:11: warning: narrowing conversion from 'float' to 'bool' [bugprone-narrowing-conversions]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:651:92: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:669:36: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:671:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:733:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:818:186: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:857:36: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:859:46: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:861:37: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:864:62: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:964:53: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:989:142: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:992:104: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:994:104: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:997:196: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1009:40: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1011:154: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1013:102: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1015:102: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1025:140: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1028:102: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1030:102: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1033:194: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1049:21: warning: implicit conversion 'int' -> 'bool' [readability-implicit-bool-conversion]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1050:21: warning: implicit conversion 'int' -> 'bool' [readability-implicit-bool-conversion]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1051:22: warning: implicit conversion 'int' -> 'bool' [readability-implicit-bool-conversion]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1205:34: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/TwoParticleCorrelations/TableProducer/longrangeMaker.cxx:1207:44: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@abmodak
abmodak enabled auto-merge (squash) September 21, 2026 15:42
@abmodak
abmodak merged commit 8135f7c into AliceO2Group:master Sep 21, 2026
13 of 15 checks passed
@vkucera

vkucera commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

@abmodak Please remember to fix reported errors before merging.

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.

4 participants