-
Notifications
You must be signed in to change notification settings - Fork 13
CMS_Z0_13TEV #2352
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
base: master
Are you sure you want to change the base?
CMS_Z0_13TEV #2352
Conversation
|
Hi @enocera a question came up The observables that are available are A_{FB} for dimuons and dielectrons, and \Delta A_{FB} for dimuons and dielectrons. Should these be in separate datasets or in the same directory implemented as different observables of the same process? At the moment I have made two directories, CMS_DY_13TEV_DIMUONS and CMS_DY_13TEV_DIELECTRONS, both containing the two observables (A_FB and \Delta A_FB), because from past datasets it seems like it was also done like this. Is this alright or should it be organised differently? Thanks! |
|
Thanks @evagroenendijk . We have the possibility of defining two different data sets, as you propose and did. Or also to define a single data set with two different distributions (one for electrons and one for muons). We don't have strong reasons to prefer one than the other, given that these are essentially uncorrelated. So I suggest that you proceed with the implementation that you've already started. |
|
Two questions/comments
|
|
@scarlehoff This is the Z forward backward asymmetry, so NC only. |
Thanks @scarlehoff and @enocera
|
|
@evagroenendijk Incidentally, I would implement only AFB (electrons, muons), not Delta AFB. |
Ok! I thought because the uncertainties are different (the pdf uncertainty cancels out mostly for \Delta A_FB), I thought it would maybe be useful? But I can take only the A_FB! |
|
The thing is that, in the SM, at parton level, Delta AFB should be zero. So this observable is better suited to study new physics than constraints on PDFs. |
Alright, clear! Thanks And shall I put them into one dataset as Juan suggested then? |
Yes, that would be desirable in light of the remark made by @scarlehoff . Also, who am I to contradict @scarlehoff ? |
|
And please change DY to Z0! |
|
Dear @evagroenendijk in view of the upcoming Morimondo meeting, I have reviewed this PR and implemented the following alterations.
|
…missing eta variable in kinematics; implemented two variants of uncertainties for full breakdown or not; removed last bin (this is the integral)
20092c6 to
b337208
Compare
|
Tests now pass, so this PR is ready for review. |
Dear @enocera Thank you! I have been on holiday for the past few weeks, and I was planning to dot the i's on this pull request in these days before the Morimondo meeting (indeed I had not finished with the metadata.yaml and the uncertainties). Thanks a lot for finishing it, I completely agree with the changes! |
scarlehoff
left a comment
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.
LGTM
Let's wait for the grids so that we have a proper comparison before merging.
Implementation of the Drell-Yan forward-backward asymmetry at high dilepton masses in proton-proton collisions at 13 TeV.