-
Notifications
You must be signed in to change notification settings - Fork 7
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
kinematic labels #2093
Closed
Closed
kinematic labels #2093
Changes from 9 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
e5293e9
ATLAS_DY
tgiani 0bb7733
small fix ATLAS_DY
tgiani de35c1a
ATLAS_PH_13TEV
tgiani 83b33bd
ATLAS_SINGLETOP_7TEV
tgiani 01145de
fix ATLAS_SINGLETOP_7TEV
tgiani 4de4294
ATLAS_SINGLETOP_8TEV
tgiani e0d3fde
ATLAS_SINGLETOP_13TEV
tgiani 9cf9469
ATLAS_WCHARM_7TEV
tgiani 80ec077
ATLAS_WJ_JET_8TEV
tgiani 69eaf8d
ATLAS_WPWM_7TEV_36PB
tgiani 06691b2
fix names accxording to vp2 conventions
tgiani f252793
ATLAS_WPWM_7TEV_46FB
tgiani c52bf08
ATLAS_WPWM_13TEV
tgiani 918354d
ATLAS_Z0_7TEV_36PB, ATLAS_Z0_7TEV_46PB
tgiani 6fed826
ATLAS_Z0_7TEV_49PB
tgiani f4ef532
Merge branch 'master' into kin_labels
enocera 01c9ada
Merge branch 'master' into kin_labels
enocera 85ffb13
ATLAS_Z0_7TEV_LOMASS
tgiani 548d671
ATLAS_Z0_8TEV_HIMASS
tgiani a1eb89f
ATLAS_Z0_8TEV_LOWMASS
tgiani 2479c98
ATLAS_Z0_13TEV
tgiani aeee36d
ATLAS_Z0J_8TEV
tgiani 2ce1698
minor edit ATLAS_Z0J_8TEV
tgiani 6d8e85b
BCDMS
tgiani 8e73ab4
CDF_Z0_1P96TEV
tgiani a61b51e
CHORUS
tgiani fad9ced
CMS_SINGLETOP
tgiani aaff3d9
CMS_TTBAR_13TEV_LJ_2016_DIF
tgiani 65380d1
CMS_WCHARM
tgiani 9068cc8
CMS_Z0J_8TEV
tgiani fd45b6f
D0_WPWM_1P96TEV
tgiani d932f5e
D0_Z0_1P96TEV
tgiani ddb1de2
DYE605
tgiani e359af9
small fix DYE605
tgiani e86f2fd
DYE866_Z0_800GEV, DYE866_Z0_800GEV_DW_RATIO
tgiani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
18 changes: 9 additions & 9 deletions
18
nnpdf_data/nnpdf_data/new_commondata/ATLAS_DY_13TEV/kinematics_TOT.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,37 +1,37 @@ | ||
bins: | ||
- k1: | ||
- zero: | ||
min: null | ||
mid: 0.0 | ||
max: null | ||
k2: | ||
MWZ: | ||
min: null | ||
mid: 80.385 | ||
max: null | ||
k3: | ||
sqrts: | ||
min: null | ||
mid: 13.0 | ||
max: null | ||
- k1: | ||
- zero: | ||
min: null | ||
mid: 0.0 | ||
max: null | ||
k2: | ||
MWZ: | ||
min: null | ||
mid: 80.385 | ||
max: null | ||
k3: | ||
sqrts: | ||
min: null | ||
mid: 13.0 | ||
max: null | ||
- k1: | ||
- zero: | ||
min: null | ||
mid: 0.0 | ||
max: null | ||
k2: | ||
MWZ: | ||
min: null | ||
mid: 91.1876 | ||
max: null | ||
k3: | ||
sqrts: | ||
min: null | ||
mid: 13.0 | ||
max: null |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use the names from
nnpdf/validphys2/src/validphys/process_options.py
Line 16 in 37f4875
(so
m_W2
instead ofMW2
...or... in this case,m_V
and we will need to add a new variable for that)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what s the procedure to add a new variable? In the case above we need
m_V
, forATLAS_PH_13TEV
we need the transverese ewnergy squareET2
and I guess there will be other cases. Is it enought to add these new variables to the_Vars
class?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. And then for the processes for which you want to add it, include it in the "accepted variables".
(if the process still doesn't exist you can either create it or leave it for later, it should not matter)