Skip to content

PWGHF: setting minimum pion pT in X candidate creator - #55

Merged
ktf merged 4 commits into
AliceO2Group:masterfrom
lucamicheletti93:myDev
Sep 11, 2021
Merged

PWGHF: setting minimum pion pT in X candidate creator#55
ktf merged 4 commits into
AliceO2Group:masterfrom
lucamicheletti93:myDev

Conversation

@lucamicheletti93

Copy link
Copy Markdown
Contributor

Adding a threshold on the pion pt in the X candidate creator

Configurable<double> d_maxdzini{"d_maxdzini", 4., "reject (if>0) PCA candidate if tracks DZ exceeds threshold"};
Configurable<double> d_minparamchange{"d_minparamchange", 1.e-3, "stop iterations if largest change of any X is smaller than this"};
Configurable<double> d_minrelchi2change{"d_minrelchi2change", 0.9, "stop iterations is chi2/chi2old > this"};
Configurable<double> minPionPtCut{"minPionPtCut", 0.15, "minimum pion pT threshold (GeV/c)"};

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.

minPionPtCut -> ptPionMin (See the naming conventions.)

@ginnocen
ginnocen marked this pull request as ready for review September 10, 2021 13:59
@ginnocen
ginnocen enabled auto-merge (squash) September 10, 2021 14:00
@ktf
ktf disabled auto-merge September 11, 2021 07:12
@ktf
ktf merged commit 8e49fca into AliceO2Group:masterSep 11, 2021
@vkucera

Copy link
Copy Markdown
Collaborator

@lucamicheletti93 Please make a PR in Run3Analysisvalidation.

alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Feb 5, 2024
alibuild added a commit to alibuild/O2Physics that referenced this pull request Apr 17, 2024
feisenhu pushed a commit that referenced this pull request Apr 17, 2024
* PWGEM: Fix compilation warnings: unused-parameter
* Please consider the following formatting changes (#55)
* Resolve conflicts
* Please consider the following formatting changes (#56)
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
christianreckziegel pushed a commit to christianreckziegel/O2Physics that referenced this pull request May 2, 2024
* PWGEM: Fix compilation warnings: unused-parameter
* Please consider the following formatting changes (AliceO2Group#55)
* Resolve conflicts
* Please consider the following formatting changes (AliceO2Group#56)
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
hahassan7 pushed a commit to hahassan7/O2Physics that referenced this pull request May 17, 2024
* PWGEM: Fix compilation warnings: unused-parameter
* Please consider the following formatting changes (AliceO2Group#55)
* Resolve conflicts
* Please consider the following formatting changes (AliceO2Group#56)
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Mar 4, 2025
[PWGUD] Please consider the following formatting changes to AliceO2Group#10309
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request May 12, 2025
[PWGCF] Please consider the following formatting changes to AliceO2Group#11114
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Jan 20, 2026
[PWGLF] Please consider the following formatting changes to AliceO2Group#14564
Sign up for freeto 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.

4 participants

@lucamicheletti93@vkucera@ginnocen@ktf