Skip to content

[PWGHF] Omegac0/Xi pT cut - #8962

Merged
fgrosa merged 13 commits into
AliceO2Group:masterfrom
FDUEnrich:master
Dec 15, 2024
Merged

[PWGHF] Omegac0/Xi pT cut#8962
fgrosa merged 13 commits into
AliceO2Group:masterfrom
FDUEnrich:master

Conversation

@FDUEnrich

@FDUEnrichFDUEnrich commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

Calculating pT of Omegac0/Xi and cut pT< setting value. Fill after pTcut distribution histos and pTcut status.

@github-actionsgithub-actionsBot added the pwghf PWG-HF label Dec 12, 2024
@github-actionsgithub-actionsBot changed the title Omegac0/Xi pT cut[PWGHF] Omegac0/Xi pT cutDec 12, 2024
@alibuildalibuild mentioned this pull request Dec 12, 2024
Merged
@FDUEnrich
Merge pull request#1from alibuild/alibot-cleanup-8962
a8bc2a6
[PWGHF] Please consider the following formatting changes to AliceO2Group#8962
fgrosa
fgrosa requested changes Dec 12, 2024
View reviewed changes

@fgrosafgrosa left a comment

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.

Hi @FDUEnrich thanks! The PR looks ok to me in general, I just have a few minor comments (see below)

Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment on lines +3238 to +3242
// a tolerance has been added to be more conservative ptMinOmegaczeroToOmegaKaLfCasc ptMinXicZeroOmegacZeroToXiPiLfCasc
Configurable<float> ptMinOmegacZeroToOmegaPiLfCasc{"ptMinOmegacZeroToOmegaPiLfCasc", 2.0, "min. pT for Omegaczero in Omega + Pi decays"};
Configurable<float> ptMinOmegaczeroToOmegaKaLfCasc{"ptMinOmegaczeroToOmegaKaLfCasc", 2.0, "min. pT for Omegaczero in Omega + Ka decays"};
Configurable<float> ptMinXicZeroOmegacZeroToXiPiLfCasc{"ptMinXicZeroOmegacZeroToXiPiLfCasc", 2.0, "min. pT for XicZeroOmegacZero in Xi + Pi decays"};
Configurable<float> ptMinXicplusLfCasc{"ptMinXicplusLfCasc", 2.0, "min. pT for Xicplus in Xi + Pi + Pi decays"};

@fgrosafgrosaDec 12, 2024

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 would put 0 as default not to change the behaviour of pp wagons

Suggested change
// a tolerance has been added to be more conservative ptMinOmegaczeroToOmegaKaLfCasc ptMinXicZeroOmegacZeroToXiPiLfCasc
Configurable<float> ptMinOmegacZeroToOmegaPiLfCasc{"ptMinOmegacZeroToOmegaPiLfCasc", 2.0, "min. pT for Omegaczero in Omega + Pi decays"};
Configurable<float> ptMinOmegaczeroToOmegaKaLfCasc{"ptMinOmegaczeroToOmegaKaLfCasc", 2.0, "min. pT for Omegaczero in Omega + Ka decays"};
Configurable<float> ptMinXicZeroOmegacZeroToXiPiLfCasc{"ptMinXicZeroOmegacZeroToXiPiLfCasc", 2.0, "min. pT for XicZeroOmegacZero in Xi + Pi decays"};
Configurable<float> ptMinXicplusLfCasc{"ptMinXicplusLfCasc", 2.0, "min. pT for Xicplus in Xi + Pi + Pi decays"};
// a tolerance has been added to be more conservative ptMinOmegaczeroToOmegaKaLfCasc ptMinXicZeroOmegacZeroToXiPiLfCasc
Configurable<float> ptMinOmegacZeroToOmegaPiLfCasc{"ptMinOmegacZeroToOmegaPiLfCasc", 0.f, "min. pT for Omegaczero in Omega + Pi decays"};
Configurable<float> ptMinOmegaczeroToOmegaKaLfCasc{"ptMinOmegaczeroToOmegaKaLfCasc", 0.f, "min. pT for Omegaczero in Omega + Ka decays"};
Configurable<float> ptMinXicZeroOmegacZeroToXiPiLfCasc{"ptMinXicZeroOmegacZeroToXiPiLfCasc", 0.f, "min. pT for XicZeroOmegacZero in Xi + Pi decays"};
Configurable<float> ptMinXicplusLfCasc{"ptMinXicplusLfCasc", 0.f, "min. pT for Xicplus in Xi + Pi + Pi decays"};

Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx Outdated
@alibuildalibuild mentioned this pull request Dec 13, 2024
Closed
@vkuceravkucera mentioned this pull request Dec 13, 2024
Closed
@vkuceravkucera reopened this Dec 13, 2024
@alibuildalibuild mentioned this pull request Dec 13, 2024
Merged
vkucera
vkucera reviewed Dec 13, 2024
View reviewed changes
Comment threadPWGHF/TableProducer/trackIndexSkimCreator.cxx
// selections have been set to run2 lambda dedicated cuts
// selections for cascade have been set to the loosest value between xi and omega
// a tolerance has been added to be more conservative
// a tolerance has been added to be more conservative ptMinOmegaczeroToOmegaKaLfCasc ptMinXicZeroOmegacZeroToXiPiLfCasc

@vkuceravkuceraDec 13, 2024

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.

Why did you add ptMinOmegaczeroToOmegaKaLfCasc ptMinXicZeroOmegacZeroToXiPiLfCasc here?

@FDUEnrichFDUEnrichDec 13, 2024

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Dear expert, thank you for comment! I added these four configurable because Fabrizio suggested doing this. We think it's better to cut pT for OmegaczeroToOmegaKa, XicZeroOmegacZeroToXiPi, OmegacZeroToOmegaPi and XicplusToXiPiPi respectively.

@fgrosafgrosaDec 13, 2024

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.

@vkucera this is for the production of derived data in Pb-Pb, for which it is unlikely that we can perform measurements to pT=0 and the size is drastically reduced if we skip the low pT

@vkuceravkuceraDec 13, 2024

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.

@FDUEnrich@fgrosa I was referring to that very line with the comment.

@FDUEnrich
Merge pull request#4from alibuild/alibot-cleanup-8962
7ea6cb3
[PWGHF] Please consider the following formatting changes to AliceO2Group#8962
@fgrosa
fgrosa merged commit 6c25c33 into AliceO2Group:masterDec 15, 2024
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Dec 20, 2024
@FDUEnrich@alibuildSabrina Hernandez
[PWGHF] Omegac0/Xi pT cut (AliceO2Group#8962)
34fb0ad
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
feisenhu pushed a commit to feisenhu/O2Physics that referenced this pull request Jan 8, 2025
@FDUEnrich@alibuild@feisenhu
[PWGHF] Omegac0/Xi pT cut (AliceO2Group#8962)
6930acd
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Feb 17, 2025
@FDUEnrich@alibuild@smaff92
[PWGHF] Omegac0/Xi pT cut (AliceO2Group#8962)
ed6551c
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkuceravkuceravkucera left review comments
@fgrosafgrosafgrosa approved these changes
@alibuildalibuildAwaiting requested review from alibuildalibuild is a code owner
@fcolamarfcolamarAwaiting requested review from fcolamarfcolamar is a code owner
@fcatalan92fcatalan92Awaiting requested review from fcatalan92fcatalan92 is a code owner
@mfagginmfagginAwaiting requested review from mfagginmfaggin is a code owner
@mmazzillimmazzilliAwaiting requested review from mmazzillimmazzilli is a code owner
@deepathomsdeepathomsAwaiting requested review from deepathomsdeepathoms is a code owner
@NicoleBastidNicoleBastidAwaiting requested review from NicoleBastidNicoleBastid is a code owner
@hahassan7hahassan7Awaiting requested review from hahassan7hahassan7 is a code owner
@jpxrkjpxrkAwaiting requested review from jpxrkjpxrk is a code owner
@apalascianoapalascianoAwaiting requested review from apalascianoapalasciano is a code owner

Assignees

No one assigned

Labels

pwghfPWG-HF

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@FDUEnrich@fgrosa@vkucera@alibuild