Skip to content

Conversation

@rashigupt
Copy link
Contributor

@rashigupt rashigupt commented Oct 5, 2025

Add BCsWithTimestamps table in MC reconstruction and mass column in electron selection to calculate invariant mass and use it in the correlation code to identify non-HFE electrons using the TOF detector.

Add DeltaEta and Deltaphi from Emcal correction table
remove some condition on Hadron table
Added TOF information to the correlation without requiring EMCAL. In addition, a downscale factor has been implemented in order to reduce memory usage during the analysis.
@rashigupt rashigupt marked this pull request as ready for review October 21, 2025 17:22
@vkucera vkucera marked this pull request as draft October 21, 2025 17:30
@rashigupt rashigupt marked this pull request as ready for review October 21, 2025 18:00
@vkucera vkucera marked this pull request as draft October 21, 2025 18:13
@rashigupt rashigupt marked this pull request as ready for review October 27, 2025 05:27
@rashigupt
Copy link
Contributor Author

Dear expert, please approve the code

jpxrk
jpxrk previously approved these changes Oct 29, 2025
@jpxrk jpxrk enabled auto-merge (squash) October 29, 2025 05:02
use BCType instead of aod::BCsWithTimestamps
auto-merge was automatically disabled October 31, 2025 07:17

Head branch was pushed to by a user without write access

Copy link
Collaborator

@vkucera vkucera left a comment

Choose a reason for hiding this comment

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

Thanks @rashigupt for addressing my comments.
Back to @jpxrk

@rashigupt
Copy link
Contributor Author

Dear expert, please merge the code

1 similar comment
@rashigupt
Copy link
Contributor Author

Dear expert, please merge the code

@jpxrk jpxrk merged commit 4262144 into AliceO2Group:master Nov 6, 2025
23 of 25 checks passed
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
…mcal.cxx: group BCsWithTimestamps in MC rec and add invariant mass column (AliceO2Group#13241)
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