Skip to content

Conversation

@bryates
Copy link
Contributor

@bryates bryates commented Jul 1, 2024

This PR ports the photons branch to histEFT.

@@ -1,5 +1,5 @@
{
"xsec": 2.2471,
"xsec": 2.9922,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

What was this change to the xsec? Was that the 33% correction? If so, we should consider moving that somewhere else instead of overwriting the xsec, just to make it more clear for future users.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Good point. Do you suggest moving that info directly inside the analysis processor where if ttgamma sample is being used, we scale the xsec by 1.33? In any case, neither of the approach should be permanent anyway cause like Kevin said, scaling cross-section won't be a physically correct thing to do.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It might be best to store that factor somewhere in a json/yml file.

anpicci added a commit that referenced this pull request Nov 11, 2025
…ions-for-custom-bins

Revert "Rebin stacked ratio plots with custom bin edges"
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.

Remove photon TopCoffea checkout

4 participants