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

Reimplement ATLAS_Z0_7TEV_36PB Dataset #2250

Merged
merged 12 commits into from
Jan 22, 2025
Prev Previous commit
Next Next commit
edited metadata
ecole41 committed Dec 18, 2024

Unverified

This user has not yet uploaded their public signing key.
commit e7b731bc18a7e70f14172490ba1cf00051ace8de
47 changes: 24 additions & 23 deletions nnpdf_data/nnpdf_data/commondata/ATLAS_Z0_7TEV_36PB/metadata.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
setname: ATLAS_Z0_7TEV_36PB
version: 1
version_comment: Port of old commondata
version: 2
version_comment: Reimplementation of old commondata
nnpdf_metadata:
nnpdf31_process: DY NC
experiment: ATLAS
@@ -18,12 +18,12 @@ implemented_observables:
description: Drell-Yan Rapidity Distribution
label: ATLAS $Z$ 7 TeV 2010
units: ''
process_type: EWK_RAP
process_type: DY_Z_Y
tables: [1]
npoints: []
ndata: 8
plotting:
kinematics_override: ewk_rap_sqrt_scale
kinematics_override: identity
dataset_label: ATLAS $Z$ 7 TeV 2010
x_label: $\left\|\eta/y\right|$
y_label: $d\sigma/dy$ (fb)
@@ -42,35 +42,36 @@ implemented_observables:
- $Z$
- $Z$
- $Z$
plot_x: k1
plot_x: abs_eta
kinematic_coverage:
- k1
- k2
- k3
- abs_eta
- m_Z2
- sqrts
kinematics:
variables:
k1:
description: Variable k1
label: k1
abs_eta:
description: Absolute value of the rapidity of the Z boson
label: abs_eta
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
label: abs_eta
label: "$y$"

Since it is rapidity and not pseudorapidity better to use y. At some point we were using eta/y to automatize a bit things (you can change also the xlabel to $\left\| y\right|$ for instance) and so, some datasets that should be Y ended up being ETA. At some point I guess we'll update them.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have just updated this

units: ''
k2:
description: Variable k2
label: k2
units: ''
k3:
description: Variable k3
label: k3
units: ''
file: kinematics_ETA.yaml
m_Z2:
description: Squared invariant mass of the Z boson
label: m_Z2
units: 'GeV^2'
sqrts:
description: Square root of the center of mass energy
label: sqrts
units: 'GeV'
file: kinematics.yaml
theory:
conversion_factor: 1.0
conversion_factor: 1.0187 #is this needed?
operation: 'null'
FK_tables:
- - ATLASWZRAP36PB-ATLAS-arXiv:1109.5141-Z0_eta34
data_uncertainties: []
data_uncertainties: [uncertainties.yaml]
variants:
legacy:
data_uncertainties:
- uncertainties_legacy_ETA.yaml
data_central: data_legacy_ETA.yaml
data_central: data_legacy_ETA.yaml
data_central: data.yaml
ported_from: ATLASZRAP36PB