Skip to content

[EventFiltering, PWGHF] Fix typo and move to HF masses#3652

Merged
fgrosa merged 2 commits into
AliceO2Group:masterfrom
fgrosa:fix_trigger
Oct 19, 2023
Merged

[EventFiltering, PWGHF] Fix typo and move to HF masses#3652
fgrosa merged 2 commits into
AliceO2Group:masterfrom
fgrosa:fix_trigger

Conversation

@fgrosa

@fgrosa fgrosa commented Oct 19, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

alibuild
alibuild previously approved these changes Oct 19, 2023

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

Auto-approving on behalf of @fgrosa.

Comment thread PWGHF/Core/PDG.h
constexpr double MassSigmaC0 = 2.45376;
constexpr double MassSigmaCPlusPlus = 2.45402;
constexpr double MassX3872 = 3.87165;
constexpr double MassXiB0 = 5.7924;

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.

Please do not edit this block by hand. It is generated by the Python script. So the script is the place to be updated and then this header should be updated with the generated output.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks! Sorry indeed it wasn't that clear to me how this worked, it should be fixed now

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.

No problem, if you have an idea, how to make the documentation clearer, let me know. Thanks for the update.

@fgrosa fgrosa disabled auto-merge October 19, 2023 15:38
@fgrosa fgrosa enabled auto-merge (squash) October 19, 2023 16:20

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

Auto-approving on behalf of @fgrosa.

@fgrosa fgrosa merged commit c518727 into AliceO2Group:master Oct 19, 2023
zconesa pushed a commit to zconesa/O2Physics that referenced this pull request Oct 27, 2023
…3652)

* Fix typo and move to HF masses

* Update python script to generate header file
chengtt0406 pushed a commit to chengtt0406/O2Physics that referenced this pull request Dec 6, 2023
…3652)

* Fix typo and move to HF masses

* Update python script to generate header file
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.

3 participants