Skip to content

enable momentum-dependent Tsallis downscaling for V0 tree - #4317

Merged
noferini merged 4 commits into
AliceO2Group:masterfrom
jezwilkinson:tsallis
Jan 15, 2024
Merged

enable momentum-dependent Tsallis downscaling for V0 tree#4317
noferini merged 4 commits into
AliceO2Group:masterfrom
jezwilkinson:tsallis

Conversation

@jezwilkinson

Copy link
Copy Markdown
Contributor

Patch to TPC PID track skimmer task to properly enable momentum-dependent downscaling via Tsallis function (method was already implemented and applied for TPC+TOF tree, but not properly enabled for V0 candidates)

@jezwilkinson

Copy link
Copy Markdown
ContributorAuthor

Hi @chiarazampolli@noferini , this is a small fix to the functionality on the PID skimmer task - could you please take a look + mark it for approval? Thanks

@noferini

Copy link
Copy Markdown
Collaborator

Hi @jezwilkinson , it is fine with me.
Did you run a quick local test to validate it?

@jezwilkinson

Copy link
Copy Markdown
ContributorAuthor

Hi @jezwilkinson , it is fine with me. Did you run a quick local test to validate it?

Thanks - Yes, I tested it locally on a few AO2Ds and it works as expected (left is before patch, right is after with same input/config)
image

@noferininoferini 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.

Fine with me

@noferini
noferini merged commit 3435d36 into AliceO2Group:masterJan 15, 2024
hahassan7 pushed a commit to hahassan7/O2Physics that referenced this pull request Feb 1, 2024
…up#4317)
* enable momentum-dependent downsampling on V0 tree
* MegaLinter fixes (AliceO2Group#7)
missing spaces around commas
* fix: consistent logic for both copies of tsallis function
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
basiach pushed a commit to basiach/O2Physics that referenced this pull request Feb 23, 2024
…up#4317)
* enable momentum-dependent downsampling on V0 tree
* MegaLinter fixes (#7)
missing spaces around commas
* fix: consistent logic for both copies of tsallis function
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
ercolessi pushed a commit to ercolessi/O2Physics that referenced this pull request Feb 28, 2024
…up#4317)
* enable momentum-dependent downsampling on V0 tree
* MegaLinter fixes (#7)
missing spaces around commas
* fix: consistent logic for both copies of tsallis function
---------
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
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.

3 participants

@jezwilkinson@noferini@alibuild