Skip to content

Conversation

@bghanley1995
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the pwgcf label Apr 1, 2025
@github-actions github-actions bot changed the title PWGCF: Identified Bf, Changed how PID is performed, added configurables to improve performance [PWGCF] Identified Bf, Changed how PID is performed, added configurables to improve performance Apr 1, 2025
bghanley1995 added a commit to bghanley1995/O2Physics that referenced this pull request Apr 1, 2025
[PWGCF] Please consider the following formatting changes to AliceO2Group#10714
@alibuild
Copy link
Collaborator

alibuild commented Apr 1, 2025

Error while checking build/O2Physics/o2 for cdb950b at 2025-04-01 10:16:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10714-slc9_x86-64/0/PWGCF/TwoParticleCorrelations/TableProducer/identifiedBfFilter.cxx:779:43: error: unused variable 'fPDG' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

[PWGCF] Please consider the following formatting changes to AliceO2Group#10714
Comment on lines +1035 to +1046
for (int sp1 = 0; sp1 < kIdBfNoOfSpecies; ++sp1) {
for (int sp2 = 0; sp2 < kIdBfNoOfSpecies; ++sp2) {
fhTrueNSigmaTPC[sp1][sp2] = new TH2F(TString::Format("fhTrueNSigmaTPC%s_%s", speciesName[sp1], speciesName[sp2]).Data(),
TString::Format("N #sigma %s from TPC vs P for generated %s;N #sigma;p (GeV/c)", speciesTitle[sp1], speciesTitle[sp2]).Data(),
48, -6, 6,
ptbins, ptlow, ptup);

fhTrueNSigmaTOF[sp1][sp2] = new TH2F(TString::Format("fhTrueNSigmaTOF%s_%s", speciesName[sp1], speciesName[sp2]).Data(),
TString::Format("N #sigma %s from TOF vs P for generated %s;N #sigma;p (GeV/c)", speciesTitle[sp1], speciesTitle[sp2]).Data(),
48, -6, 6,
ptbins, ptlow, ptup);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why two species indexes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to examine how the smearing effect on the N sigma values of TPC cause contamination when I run PID. When I fill these histograms, the first species index is species that I am measuring N Sigma for, the other is the generated species as determined by PDG value. I also use the plot corresponding to the generated particle species (eg, N Sigma pion for generated pions) to get my corrections for the smearing.

@victor-gonzalez victor-gonzalez merged commit ed2b824 into AliceO2Group:master Apr 1, 2025
13 of 14 checks passed
louisemillot pushed a commit to louisemillot/O2Physics that referenced this pull request Apr 8, 2025
…les to improve performance (AliceO2Group#10714)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
njacazio pushed a commit that referenced this pull request Apr 11, 2025
…les to improve performance (#10714)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Apr 15, 2025
…les to improve performance (AliceO2Group#10714)

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
…les to improve performance (AliceO2Group#10714)

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

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

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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants