Skip to content

[PWGCF] FemtoUniverse -- Add helicity angle analysis in lambda decay.#11254

Merged
prchakra merged 9 commits into
AliceO2Group:masterfrom
a-m-andrushko:cosine_theta
Jun 6, 2025
Merged

[PWGCF] FemtoUniverse -- Add helicity angle analysis in lambda decay.#11254
prchakra merged 9 commits into
AliceO2Group:masterfrom
a-m-andrushko:cosine_theta

Conversation

@a-m-andrushko

Copy link
Copy Markdown
Contributor

Helicity angle theta is calculated between lambda and its daughter (proton) in PRF of lambda.

@github-actions

github-actions Bot commented May 19, 2025

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title PWGCF: FemtoUniverse -- Add helicity angle analysis in lambda decay. [PWGCF] FemtoUniverse -- Add helicity angle analysis in lambda decay. May 19, 2025
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 7c7177f at 2025-05-20 14:53:

No log files found

Full log here.

@victor-gonzalez

Copy link
Copy Markdown
Collaborator

Hi @vkucera just to let you know about the MegaLinter cppcheck error which seems to be another false positive
Regarding the gitleaks one regarding thekeys, I thought it was already addressed. Wasn't it?

@a-m-andrushko the linter errors regarding TLorentzVector and TDatabasePDG.h have impacts on execution performance. Please, address them in further iterations

@a-m-andrushko

Copy link
Copy Markdown
Contributor Author

@victor-gonzalez, could you review and accept the changes, please?
The remaining errors will be addressed in further PRs.

@vkucera

vkucera commented May 26, 2025

Copy link
Copy Markdown
Collaborator

Hi @vkucera just to let you know about the MegaLinter cppcheck error which seems to be another false positive Regarding the gitleaks one regarding thekeys, I thought it was already addressed. Wasn't it?

@a-m-andrushko the linter errors regarding TLorentzVector and TDatabasePDG.h have impacts on execution performance. Please, address them in further iterations

Both issues have been fixed. The problem is that the PR branch is 1770 commits behind the upstream master.

@victor-gonzalez

Copy link
Copy Markdown
Collaborator

Hi @vkucera just to let you know about the MegaLinter cppcheck error which seems to be another false positive Regarding the gitleaks one regarding thekeys, I thought it was already addressed. Wasn't it?
@a-m-andrushko the linter errors regarding TLorentzVector and TDatabasePDG.h have impacts on execution performance. Please, address them in further iterations

Both issues have been fixed. The problem is that the PR branch is 1770 commits behind the upstream master.

Oh! Wow!
@a-m-andrushko Could you please rebase your changes to the latest version of O2Physics?!!!
@vkucera Thanks!!!

@a-m-andrushko

Copy link
Copy Markdown
Contributor Author

Hi @vkucera just to let you know about the MegaLinter cppcheck error which seems to be another false positive Regarding the gitleaks one regarding thekeys, I thought it was already addressed. Wasn't it?
@a-m-andrushko the linter errors regarding TLorentzVector and TDatabasePDG.h have impacts on execution performance. Please, address them in further iterations

Both issues have been fixed. The problem is that the PR branch is 1770 commits behind the upstream master.

Oh! Wow! @a-m-andrushko Could you please rebase your changes to the latest version of O2Physics?!!! @vkucera Thanks!!!

@victor-gonzalez, indeed, it was my mistake, I did not synchronise the branch before committing, sorry; now there are no issues with MegaLinter.

Comment on lines +112 to +120
auto pdgDB = TDatabasePDG::Instance();
auto protonMass = pdgDB->GetParticle(confPDGCodePositiveChild)->Mass();
auto pionMass = pdgDB->GetParticle(confPDGCodeNegativeChild)->Mass();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there any reason for, now that the O2DatabasePDG service has been instantiated L.44, not using it for getting the particle masses as pdg->Mass(confPDGCodePositiveChild) for instance?

@vkucera vkucera marked this pull request as draft June 6, 2025 15:24
@vkucera

vkucera commented Jun 6, 2025

Copy link
Copy Markdown
Collaborator

@a-m-andrushko You did not rebase properly. You have unresolved merge conflicts in the commit.

@a-m-andrushko a-m-andrushko marked this pull request as ready for review June 6, 2025 15:35
@a-m-andrushko

Copy link
Copy Markdown
Contributor Author

@a-m-andrushko You did not rebase properly. You have unresolved merge conflicts in the commit.

@vkucera, I cannot see this problem from my side, where can you see conflicts?

Also, I might not have rebased properly indeed, because the errors that I had been facing before the rebase still remain. I did 'git fetch upstream' and 'git rebase upstream/master'.

@vkucera

vkucera commented Jun 6, 2025

Copy link
Copy Markdown
Collaborator

I saw them in 117b5e7. You fixed them in cbbe847

@vkucera

vkucera commented Jun 6, 2025

Copy link
Copy Markdown
Collaborator

Now your branch is again very obsolete.
Please do not make your PR ready for review if it has obvious errors, as reported by the linters.

@vkucera vkucera marked this pull request as draft June 6, 2025 16:21
@a-m-andrushko

a-m-andrushko commented Jun 6, 2025

Copy link
Copy Markdown
Contributor Author

Now your branch is again very obsolete. Please do not make your PR ready for review if it has obvious errors, as reported by the linters.

@vkucera, I rebased properly this time, MegaLinter errors are fixed; when it comes to O2 linter, these errors are a bit complex and will be fixed in future PRs.

@vkucera vkucera marked this pull request as ready for review June 6, 2025 16:59
@prchakra prchakra merged commit cc91e39 into AliceO2Group:master Jun 6, 2025
11 of 12 checks passed
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
@a-m-andrushko a-m-andrushko deleted the cosine_theta branch August 3, 2025 12:00
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.

5 participants