Skip to content

PWGHF: add config for Pb-Pb MCs without pt hard bins for signal#1743

Merged
stefanopolitano merged 2 commits intoAliceO2Group:masterfrom
fgrosa:ptHardBins
Sep 10, 2024
Merged

PWGHF: add config for Pb-Pb MCs without pt hard bins for signal#1743
stefanopolitano merged 2 commits intoAliceO2Group:masterfrom
fgrosa:ptHardBins

Conversation

@fgrosa
Copy link
Contributor

@fgrosa fgrosa commented Sep 10, 2024

No description provided.

@github-actions
Copy link

github-actions bot commented Sep 10, 2024

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass3
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0

@fgrosa fgrosa added async-2023-pbpb-apass4 Request porting to async-2023-pbpb-apass4 async-2023-pp-apass4 Request porting to async-2023-pp-apass4 async-2024-pp-apass1 Request porting to async-2024-pp-apass1 async-2022-pp-apass7 Request porting to async-2022-pp-apass7 labels Sep 10, 2024
stefanopolitano
stefanopolitano approved these changes Sep 10, 2024
View reviewed changes
Copy link
Contributor

@stefanopolitano stefanopolitano left a comment

Choose a reason for hiding this comment

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

Hi @fgrosa! Thank you for the PR. I don't have any comments. Approved!

@stefanopolitano stefanopolitano merged commit c0c21b2 into AliceO2Group:master Sep 10, 2024
alcaliva pushed a commit that referenced this pull request Sep 27, 2024
@fgrosa @alcaliva
PWGHF: add config for Pb-Pb MCs without pt hard bins for signal (#1743)
4598f7a 
* PWGHF: add config for Pb-Pb MCs without pt hard bins for signal

* Add missing config

(cherry picked from commit c0c21b2)
@alcaliva alcaliva removed the async-2023-pbpb-apass4 Request porting to async-2023-pbpb-apass4 label Sep 27, 2024
@alcaliva
Copy link
Collaborator

alcaliva commented Sep 27, 2024

this PR is now included in the tag for the Pb-Pb apass4 that is being built. Label removed

alcaliva pushed a commit that referenced this pull request Sep 27, 2024
@fgrosa @alcaliva
PWGHF: add config for Pb-Pb MCs without pt hard bins for signal (#1743)
57998d3 
* PWGHF: add config for Pb-Pb MCs without pt hard bins for signal

* Add missing config

(cherry picked from commit c0c21b2)
@alcaliva alcaliva removed the async-2024-pp-apass1 Request porting to async-2024-pp-apass1 label Sep 27, 2024
@alcaliva
Copy link
Collaborator

alcaliva commented Oct 17, 2024

ciao @fgrosa, this PR gave me conflicts in the creation of the tag for async-2022-pp-apass7. I started from the tag async-v1-01-02d. Do you perhaps have any clue why this didn't work?

alcaliva pushed a commit that referenced this pull request Oct 18, 2024
@fgrosa @alcaliva
PWGHF: add config for Pb-Pb MCs without pt hard bins for signal (#1743)
45bf60e 
* PWGHF: add config for Pb-Pb MCs without pt hard bins for signal

* Add missing config

(cherry picked from commit c0c21b2)
@alcaliva alcaliva removed the async-2023-pp-apass4 Request porting to async-2023-pp-apass4 label Oct 18, 2024
@alcaliva
Copy link
Collaborator

alcaliva commented Oct 18, 2024

it worked for pp apass4 2023

@chiarazampolli chiarazampolli mentioned this pull request Dec 5, 2024
Merged
@chiarazampolli
Copy link
Collaborator

chiarazampolli commented Dec 5, 2024

Ported to async-v1-01-02f.

@chiarazampolli chiarazampolli removed the async-2022-pp-apass7 Request porting to async-2022-pp-apass7 label Dec 5, 2024
chiarazampolli pushed a commit that referenced this pull request Dec 5, 2024
@fgrosa @chiarazampolli
PWGHF: add config for Pb-Pb MCs without pt hard bins for signal (#1743)
e373309 
* PWGHF: add config for Pb-Pb MCs without pt hard bins for signal

* Add missing config

(cherry picked from commit c0c21b2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stefanopolitano stefanopolitano stefanopolitano approved these changes

@sawenzel sawenzel Awaiting requested review from sawenzel

@chiarazampolli chiarazampolli Awaiting requested review from chiarazampolli

@benedikt-voelkel benedikt-voelkel Awaiting requested review from benedikt-voelkel

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@fgrosa @alcaliva @chiarazampolli @stefanopolitano