Skip to content

Conversation

@pstahlhu
Copy link
Contributor

@pstahlhu pstahlhu commented Jul 7, 2025

@JaeYoonCHO @sh-lim @klsmith15k

  • Fill selection counter histogram(s) separately for matched signal candidates and reconstructed background candidates in new processMc function in candidateSelectorXicToXiPiPi.cxx
  • Restructure full/lite tables for KF workflow in treeCreatorXicToXiPiPi.cxx
  • Fill the residual table only for matched candidates, i.e. avoid filling with dummy values for background candidates

@github-actions
Copy link

github-actions bot commented Jul 7, 2025

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

@pstahlhu pstahlhu marked this pull request as draft July 7, 2025 14:46
@pstahlhu pstahlhu marked this pull request as ready for review July 7, 2025 14:49
@pstahlhu
Copy link
Contributor Author

pstahlhu commented Jul 7, 2025

Hi @vkucera, thank you for your comments! I have implemented them now. I also changed the data type of the sign column and corresponding variables to int8_t as for the other flags.

@vkucera vkucera enabled auto-merge (squash) July 7, 2025 16:49
@vkucera vkucera merged commit ae7e00c into AliceO2Group:master Jul 7, 2025
14 checks passed
jpxrk pushed a commit to jpxrk/O2Physics that referenced this pull request Jul 16, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jul 18, 2025
JimunLee pushed a commit to JimunLee/O2Physics that referenced this pull request Jul 22, 2025
vojmach pushed a commit to vojmach/O2Physics that referenced this pull request Jul 23, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
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.

2 participants