Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding folder CAT23001 with datacards used in the paper #865

Merged
merged 3 commits into from Dec 14, 2023

Conversation

nucleosynthesis
Copy link
Contributor

As it says on the tin

Copy link
Collaborator

@kcormi kcormi left a comment

Choose a reason for hiding this comment

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

Thanks Nick, looks good to me, one small comment.

Would you like to merge this in now, or keep it open and update as the paper proceeds? Either way is fine for me.

Comment on lines +14 to +15
deltaS lnN 1.20 - 20\% uncertainty on signal
deltaB lnN - 1.50 50\% uncertainty on background
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if its worth adding a '#' in front of the comments for these lines to make it clear they are comments (despite the fact that they run fine).

@nucleosynthesis
Copy link
Contributor Author

I think worth keeping this open until the paper is ready in case we want to make changes from there

@nucleosynthesis nucleosynthesis merged commit b9538db into main Dec 14, 2023
6 checks passed
@nucleosynthesis nucleosynthesis deleted the nckw_adding_CAT23001_cards branch February 15, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants