Skip to content

Implementation of CMS_Z0J_13TeV - general cleanup#2468

Merged
enocera merged 13 commits into
masterfrom
CMS_Z0J_13TEV_cleanup
May 13, 2026
Merged

Implementation of CMS_Z0J_13TeV - general cleanup#2468
enocera merged 13 commits into
masterfrom
CMS_Z0J_13TEV_cleanup

Conversation

@enocera
Copy link
Copy Markdown
Contributor

@enocera enocera commented May 12, 2026

This PR replaces and supersedes part of #2360 . This PR contains the cleaned-up implementation of the CMS ZpT 13 TeV data set. Comments in #2360 relevant to this data set are taken into account For the time being, standard cuts due to lack of resummation and of EW corrections are implemented.

The values of the chi2 are as follows (with NNPDF40_nnlo_as_01180 as input PDF set)
Exp chi2: 1.354
Exp+th chi2: 0.7604
Exp+th+pdf chi2: 0.7573

Here are the data/theory comparison plots. For once, all seems good to me.
plot_bin4
plot_bin3
plot_bin2
plot_bin1

@enocera enocera changed the title Implementation of CMS_z0J_13TeV - general cleanup Implementation of CMS_Z0J_13TeV - general cleanup May 12, 2026
@enocera enocera requested a review from scarlehoff May 12, 2026 09:48
@scarlehoff
Copy link
Copy Markdown
Member

I was about to ask you about #2360 you read my mind!

@enocera enocera mentioned this pull request May 12, 2026
rule: "pT <= 170"

- dataset: CMS_Z0J_13TEV_PT-M
reason: Remove first invariant mass bin which is completely off.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did we have an idea why?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No I don't. And I don't see any reason why the NNLOjet runcard should be wrong. At this point life is too short, and I would just impose a cut, as I did. If you want I can add "which is completely off for unknown reasons".

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes please.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

description: "Z boson mass squared"
label: '$m^2_{\ell \ell}$'
units: "GeV^2"
file: kinematics.yaml
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think the m_ll variable is actually squared.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're totally right. Fixed. I am regenerating the data-theory comparison plots.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Here are the plots with the correct label.
plot_bin1
plot_bin2
plot_bin3
plot_bin4

Copy link
Copy Markdown
Member

@scarlehoff scarlehoff left a comment

Choose a reason for hiding this comment

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

Please check hte m_ll comment. Looking at kinematics.yaml they look like GeVs

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented May 12, 2026

Please check hte m_ll comment. Looking at kinematics.yaml they look like GeVs

You're right. These are GeV. Just look at the Z-mass peak in the data-theory comparison plots. The square does not make sense at all. Apologies.

@scarlehoff
Copy link
Copy Markdown
Member

fwiw, I would actually prefer to keep m_ll and use that in process options (so it needs to be changed to accept it)

It's easier to understand the numbers/plots/cuts that way (but no strong opinion).

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented May 12, 2026

fwiw, I would actually prefer to keep m_ll and use that in process options (so it needs to be changed to accept it)

It's easier to understand the numbers/plots/cuts that way (but no strong opinion).

OK, so let me do that.

@enocera
Copy link
Copy Markdown
Contributor Author

enocera commented May 13, 2026

@scarlehoff If you're happy with the changes, I'll proceed to merge this PR, and then rebase #2469 .

@enocera enocera merged commit ca2d3fa into master May 13, 2026
20 of 22 checks passed
@enocera enocera deleted the CMS_Z0J_13TEV_cleanup branch May 13, 2026 09:03
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.

2 participants