Skip to content

Conversation

@lucamicheletti93
Copy link
Collaborator

Add dimuon channel to X task

Copy link
Collaborator

@vkucera vkucera left a comment

Choose a reason for hiding this comment

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

The selector is quite obsolete. Please have a look at the newer selectors and update it accordingly, not necessarily in this PR, though.

Comment on lines +783 to +787
float massX = 3.872; // replace this with: "RecoDecay::getMassPDG(9920443)" when pdg is added
template <typename T>
auto CtX(const T& candidate)
{
return candidate.ct(massX);
Copy link
Collaborator

@vkucera vkucera Sep 27, 2021

Choose a reason for hiding this comment

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

Get the mass of X and Jpsi from RecoDecay using the PDG codes.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For the X this cannot be done because the X is not in the database
[FATAL] Cannot find particle mass for PDG code 9920443

Copy link
Collaborator

@vkucera vkucera Sep 27, 2021

Choose a reason for hiding this comment

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

It is #117

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please fix this in the next PR together with the update of the selector.

@ginnocen ginnocen marked this pull request as ready for review September 27, 2021 15:57
@ginnocen ginnocen self-requested a review as a code owner September 27, 2021 15:57
@vkucera vkucera enabled auto-merge (squash) September 27, 2021 17:25
@vkucera vkucera merged commit 983f119 into AliceO2Group:master Sep 27, 2021
@lucamicheletti93 lucamicheletti93 deleted the myDev branch September 29, 2021 19:28
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request May 6, 2024
Please consider the following formatting changes to AliceO2Group#5893
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.

2 participants