Skip to content

PWGFH: Adding Lc in pKpi in the analysis workflow#4727

Merged
jgrosseo merged 1 commit into
AliceO2Group:devfrom
DelloStritto:dev
Oct 30, 2020
Merged

PWGFH: Adding Lc in pKpi in the analysis workflow#4727
jgrosseo merged 1 commit into
AliceO2Group:devfrom
DelloStritto:dev

Conversation

@DelloStritto

Copy link
Copy Markdown
Contributor
  • First version of the Lc pKpi selection
  • First version of the Lc pKpi task
  • Insertion of the Lc pKpi analysis in the workflow

@ginnocen @vkucera @nzardosh

@jgrosseo jgrosseo merged commit ca5e1a1 into AliceO2Group:dev Oct 30, 2020

namespace o2::aod
{
namespace hf_selcandidate_lc

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.

hf_selcandidate should be renamed to hf_selcandidate_d0.

{
DECLARE_SOA_COLUMN(IsSelLc, isSelLc, int);
} // namespace hf_selcandidate_lc
DECLARE_SOA_TABLE(HFSelLcCandidate, "AOD", "HFSELLcCAND", hf_selcandidate_lc::IsSelLc);

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.

The descriptor should be in capitals.

Comment on lines +304 to +308
template <typename T>
auto InvMassLc(const T& candidate)
{
return candidate.m(array{RecoDecay::getMassPDG(kProton), RecoDecay::getMassPDG(kKPlus), RecoDecay::getMassPDG(kPiPlus)});
}

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.

I think there should be a second version of this with proton and pion hypotheses swapped.

Comment on lines +65 to +74
int getpTBin(T candpT)
{
double pTBins[npTBins + 1] = {0, 1., 2., 3., 4., 5., 6., 8., 12., 24., 36.};
for (int i = 0; i < npTBins; i++) {
if (candpT >= pTBins[i] && candpT < pTBins[i + 1]) {
return i;
}
}
return -1;
}

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.

There was an update of the D0 selector recently. #4692
Have a look and update this task accordingly, please.

Comment on lines +199 to +207
} else {
return nSigma = 100; //arbitarily large value
}
if (nSigma < nSigmaCut) {
return true;
} else {
return false;
}
}

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.

Fix this as in #4692

Comment on lines +227 to +233
return nSigma = 100; //arbitarily large value
}
if (nSigma < nSigmaCut) {
return true;
} else {
return false;
}

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.

Fix this as in #4692

Comment on lines +301 to +304
proton = -1;
kMinus = -1;
piPlus = -1;

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.

Fix this as in #4692

Comment on lines +327 to +329
proton = selectionPID(trackPos1, 2212);
kMinus = selectionPID(trackNeg1, 321);
piPlus = selectionPID(trackPos2, 211);

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.

Fix this as in #4692

//OutputObj<TH1F> hdca{TH1F("hdca", "3-prong candidates;prongs DCA to prim. vertex (cm);entries", 100, -1., 1.)};
OutputObj<TH1F> hdca2{TH1F("hdca2", "3-prong candidates;prongs DCA to sec. vertex (cm);entries", 100, 0., 1.)};

Configurable<int> d_selectionFlagLc{"d_selectionFlagLc", 1, "Selection Flag for Lc"};

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.

There should be two hypotheses.

DelloStritto pushed a commit to DelloStritto/AliceO2 that referenced this pull request Oct 30, 2020
tklemenz pushed a commit to tklemenz/AliceO2 that referenced this pull request Nov 12, 2020
tklemenz pushed a commit to tklemenz/AliceO2 that referenced this pull request Nov 12, 2020
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants