Skip to content

[PWGLF] added second option for mass calculation #11459

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.

Sign up for GitHub

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

Jump to bottom
Merged
maciacco merged 31 commits into from
Jun 9, 2025
Merged

[PWGLF] added second option for mass calculation #11459

maciacco merged 31 commits into from
Jun 9, 2025

Conversation

@mherzer28
Copy link
Contributor

@mherzer28 mherzer28 commented Jun 3, 2025

No description provided.

mherzer28 and others added 22 commits April 14, 2025 14:48
@mherzer28
added tof mass cut
b521cd7
@alibuild
Please consider the following formatting changes
fc4591c
@mherzer28
Merge pull request #6 from alibuild/alibot-cleanup-10844
f1d3b5e 
Please consider the following formatting changes to AliceO2Group#10844
@mherzer28
Merge branch 'AliceO2Group:master' into master
6b377c2
@mherzer28
Merge branch 'AliceO2Group:master' into master
47f8791
@mherzer28
Merge branch 'AliceO2Group:master' into master
0cafff3
@mherzer28
Merge branch 'AliceO2Group:master' into master
177676e
@mherzer28
rebase
3081997
@mherzer28
minor linter fix
efb5749
@mherzer28
linter fix
cbfa163
@mherzer28
linter fix 2
4199ba5
@mherzer28
fix
8f73d71
@mherzer28
fix error
6771246
@mherzer28
mass and bethe bloch calculation
db54e59
@mherzer28
Merge branch 'AliceO2Group:master' into master
3ec9afe
@mherzer28
linter fix
fe0454c
@alibuild
Please consider the following formatting changes
2a2a5f0
@mherzer28
Merge pull request #8 from alibuild/alibot-cleanup-11385
4a5bc6d 
Please consider the following formatting changes to AliceO2Group#11385
@mherzer28
fix
0537ba7
@alibuild
Please consider the following formatting changes
e8a679a
@mherzer28
Merge pull request #9 from alibuild/alibot-cleanup-11385
7baf4d7 
Please consider the following formatting changes to AliceO2Group#11385
@mherzer28
added second option for mass calculation
9123320
@github-actions github-actions bot added the pwglf label Jun 3, 2025
@github-actions github-actions bot changed the title added second option for mass calculation [PWGLF] added second option for mass calculation Jun 3, 2025
@alibuild alibuild mentioned this pull request Jun 3, 2025
Merged
@github-actions
Copy link

github-actions bot commented Jun 3, 2025 •
edited
Loading

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

@alibuild alibuild mentioned this pull request Jun 3, 2025
Closed
@alibuild
Copy link
Collaborator

alibuild commented Jun 3, 2025

Error while checking build/O2Physics/o2 for 02ec462 at 2025-06-03 22:55:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11459-slc9_x86-64/0/PWGLF/TableProducer/Nuspex/trHeAnalysis.cxx:38:10: fatal error: PWGLF/DataModel/PIDTOFGeneric.h: No such file or directory
ninja: build stopped: subcommand failed.

Full log here.

@mherzer28 mherzer28 closed this Jun 4, 2025
@mherzer28 mherzer28 reopened this Jun 9, 2025
@alibuild
Copy link
Collaborator

alibuild commented Jun 9, 2025

Error while checking build/O2Physics/o2 for 18cfc2e at 2025-06-09 11:36:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11459-slc9_x86-64/0/PWGLF/TableProducer/Nuspex/trHeAnalysis.cxx:144:42: error: redeclaration of 'constexpr const int Particle::NNumBetheParams'
/sw/SOURCES/O2Physics/11459-slc9_x86-64/0/PWGLF/TableProducer/Nuspex/trHeAnalysis.cxx:709:32: error: 'CInCmPs' was not declared in this scope; did you mean 'CInCmPS'?
/sw/SOURCES/O2Physics/11459-slc9_x86-64/0/PWGLF/TableProducer/Nuspex/trHeAnalysis.cxx:705:23: error: unused variable 'CInCmPS' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@maciacco maciacco merged commit cb9ff1b into AliceO2Group:master Jun 9, 2025
12 of 13 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
@mherzer28 @alibuild @jinhyunni
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
4e8cc74 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jun 11, 2025
@mherzer28 @alibuild
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
5f1422a 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
@mherzer28 @alibuild
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
6950c79 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
lietava pushed a commit to lietava/O2Physics that referenced this pull request Jun 14, 2025
@mherzer28 @alibuild @lietava
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
be9d88f 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
@mherzer28 @alibuild
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
6427f61 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
alibuild added a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@mherzer28 @alibuild @CesarOmarRA
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
9f1f6d1 
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@mherzer28 @alibuild @jloemker
[PWGLF] added second option for mass calculation (AliceO2Group#11459)
0128cc5 
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

Reviewers

@maciacco maciacco maciacco approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner

@fmazzasc fmazzasc Awaiting requested review from fmazzasc fmazzasc is a code owner

@chiarapinto chiarapinto Awaiting requested review from chiarapinto

Assignees

No one assigned

Labels

pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mherzer28 @alibuild @maciacco