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] Update LFResonanceTables and Resonance Module Initialization Code #8900

Merged
merged 5 commits into from
Dec 10, 2024

Conversation

BongHwi
Copy link
Collaborator

@BongHwi BongHwi commented Dec 10, 2024

  1. Enhancements to LFResonanceTables.h

    • Added index columns for Collisions, Tracks, V0s, and Cascades using DECLARE_SOA_INDEX_COLUMN_FULL.
    • The addition of index columns aims to improve interoperability with other frameworks, allowing for clearer reference and integration across different data models.
    • Renamed and standardized certain column names (e.g., MothersIdMotherId, CascIndicescascadeIndices).
    • Incorporated collision and index columns into ResoCollisions, ResoMCCollisions, ResoSpheroCollisions, and ResoEvtPlCollisions tables to improve reference handling.
    • Applied consistent naming changes (MotherId, DaughterID) in MC-related tables.
  2. New Resonance Module Initialization Workflow (resonance-module-initializer)

    • Added resonanceModuleInitializer.cxx.
    • Introduced indexing and standardized field setup for collisions and daughter particles (tracks, V0s, cascades).
    • Included MC-specific information (mothers, daughters, PDG codes).
    • Improved logic for CCDB access and magnetic field (dBz) initialization.
    • Separated Run2/Run3 and MC data workflows for more streamlined initialization.
  3. Additional Modifications

Please consider the following formatting changes to AliceO2Group#8900
njacazio
njacazio previously approved these changes Dec 10, 2024
Copy link
Collaborator

@njacazio njacazio left a comment

Choose a reason for hiding this comment

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

Super, thanks a lot @BongHwi

@BongHwi BongHwi merged commit 06a4819 into AliceO2Group:master Dec 10, 2024
12 of 13 checks passed
@BongHwi BongHwi deleted the resoDataUpdate branch December 10, 2024 19:12
@BongHwi BongHwi restored the resoDataUpdate branch December 10, 2024 19:12
@BongHwi BongHwi deleted the resoDataUpdate branch December 10, 2024 19:12
Archita-Dash pushed a commit to Archita-Dash/O2Physics that referenced this pull request Dec 11, 2024
…Code (AliceO2Group#8900)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Dec 20, 2024
…Code (AliceO2Group#8900)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
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