Skip to content
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

[PWGLF] Refinements and Enhancements in Resonance Initialization and Module Setup #8974

Merged
merged 3 commits into from
Dec 19, 2024

Conversation

BongHwi
Copy link
Collaborator

@BongHwi BongHwi commented Dec 13, 2024

  1. Resolved Duplicate V0 Filling:

    • Removed duplicate calls to fillV0s in resonanceInitializer.cxx for efficient processing.
  2. Added New Processes:

    • Introduced processSpherocity and processEventPlane methods in resonanceModuleInitializer.cxx:
      • Spherocity processing includes configurable track-based spherocity calculations.
      • Event plane processing incorporates event plane resolutions with flexible configurations.
  3. Enhanced Filters and Configurations:

    • Simplified the PID filter definitions by removing commented and redundant code.
    • Introduced spherocity-specific configuration options:
      • cfgTrackSphMin and cfgTrackSphDef.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for c07d0e0 at 2024-12-13 01:40:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/8974-slc9_x86-64/0/PWGLF/TableProducer/Resonances/resonanceModuleInitializer.cxx:610:156: error: unused parameter 'tracks' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@njacazio njacazio enabled auto-merge (squash) December 19, 2024 09:46
@njacazio njacazio merged commit 240d27a into AliceO2Group:master Dec 19, 2024
13 checks passed
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Dec 20, 2024
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.

3 participants