-
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
Reimplement ATLAS_WPWM_7TEV_46FB #2202
Conversation
Compatability Checks:
|
Hi @ecole41, thanks this is very nice!
|
nnpdf_data/nnpdf_data/commondata/ATLAS_WPWM_7TEV_46FB/filter.py
Outdated
Show resolved
Hide resolved
|
nnpdf_data/nnpdf_data/commondata/ATLAS_WPWM_7TEV_46FB/metadata.yaml
Outdated
Show resolved
Hide resolved
nnpdf_data/nnpdf_data/commondata/ATLAS_WPWM_7TEV_46FB/metadata.yaml
Outdated
Show resolved
Hide resolved
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.
This seems ok and ready to merge up to the UNCORR.1
bug, however you have to fix the conflicts with master.
In this case, what you added to commondataparser.py
and process_options.py
was already added to master in the meantime I believe so the easiest thing would be to rebase by picking up only the changes to the ATLAS_WMWP...
folder
nnpdf_data/nnpdf_data/commondata/ATLAS_WPWM_7TEV_46FB/uncertainties.yaml
Outdated
Show resolved
Hide resolved
The failing tests is because of changes in |
"SINGLETOP": ("$y$", "$m_t^2 (GeV^2)$", "$\\sqrt{s} (GeV)$"), | ||
"DY_MLL": ("$M_{ll} (GeV)$", "$M_{ll}^2 (GeV^2)$", "$\\sqrt{s} (GeV)$"), | ||
"DY_W_ETA": ("$abs_\\eta$", "$M_W^2 (GeV^2)$", "$\\sqrt{s} (GeV)$"), |
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.
I am not sure if this is a correct edit, the first kinematic is abs_eta for this specific dataset but I am not sure for other DY_W_ETA processes
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.
This whole thing will go away soon as soon as all datasets are ported since the labels are taken from the metadata, so it's not important.
This pull request introduces a new filtering module for the ATLAS WPMP 7TeV 47FB dataset, along with several supporting utility functions and updates to various data files. The most important changes include the addition of filtering functions, utility functions for data extraction, and updates to metadata and raw data files.
Old vs New Data Comparison
https://vp.nnpdf.science/GCCc3EPrTr-G7SgQBTeTbA==/
Compatibility Checks