Skip to content

Conversation

@zhangbiao-phy
Copy link
Collaborator

@zhangbiao-phy zhangbiao-phy commented May 14, 2025

For convenience in doing the mixed event locally, I added the collision index to the table

  • Separate into 3 tables: charm, track, and collision
  • Add eta and phi into the table --> calculate k*, kT, mT locally
  • Remove table filling in the mix-event function (if the test is smooth, this part will be deleted later)

@github-actions
Copy link

github-actions bot commented May 14, 2025

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

@github-actions github-actions bot changed the title [PWGHF, PWGCF] add the collision index into the table [PWGHF,PWGCF] add the collision index into the table May 14, 2025
@zhangbiao-phy zhangbiao-phy changed the title [PWGHF,PWGCF] add the collision index into the table [PWGHF,PWGCF] Split same-event table and add the collision infos into lcp femto task May 15, 2025
@zhangbiao-phy zhangbiao-phy marked this pull request as ready for review May 15, 2025 16:43
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 902f26d at 2025-05-15 19:43:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
/sw/SOURCES/O2Physics/11191-slc9_x86-64/0/PWGHF/HFC/Tasks/taskCharmHadronsFemtoDream.cxx:369:15: error: variable 'chargeTrack' set but not used [-Werror=unused-but-set-variable]
ninja: build stopped: subcommand failed.

Full log here.

fdhf::OriginMcRec);

DECLARE_SOA_TABLE(FDResultsHFTrkInfo, "AOD", "FDRESULTSHFTRKINFO", //! table to store results for HF femtoscopy
DECLARE_SOA_TABLE(FDResultsTrk, "AOD", "FDRESULTSTRK", //! table to store results for HF femtoscopy
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please, keep HF as before in the table declaration

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, Fixed

femtodreamparticle::TPCNSigmaPr,
femtodreamparticle::TOFNSigmaPr);

DECLARE_SOA_TABLE(FDResultsColl, "AOD", "FDRESULTSCOLL", //! table to store results for HF femtoscopy
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please, incorporate HF in the table declaration

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

@zhangbiao-phy zhangbiao-phy enabled auto-merge (squash) May 16, 2025 07:56
Copy link
Collaborator

@alibuild alibuild left a comment

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 @zhangbiao-phy.

@zhangbiao-phy zhangbiao-phy merged commit 7d40bc8 into AliceO2Group:master May 16, 2025
12 of 13 checks passed
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request May 17, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
alibuild added a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
… lcp femto task (AliceO2Group#11191)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
… lcp femto task (AliceO2Group#11191)

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.

4 participants