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

CDFZRAP with last two grids combined #33

Closed
cschwan opened this issue Jan 22, 2021 · 2 comments
Closed

CDFZRAP with last two grids combined #33

cschwan opened this issue Jan 22, 2021 · 2 comments
Assignees

Comments

@cschwan
Copy link
Contributor

cschwan commented Jan 22, 2021

Commit 1eb397d was supposed to go in a different branch, but I forgot to switch to that branch. However, I didn't overwrite anything so it shouldn't be too bad.

The commit contains the updated root file and the C++ program which combines the last two bins. Performing a convolution with NNPDF31_nlo_as_0118_luxqed before the combination gives

...
bin #26: 3.054914e+00 [pb/GeV] or 3.054914e-01 [pb]
bin #27: 1.433284e+00 [pb/GeV] or 1.433284e-01 [pb]
bin #28: 5.821576e-01 [pb/GeV] or 5.821576e-02 [pb]

and after the combination

...
bin #26: 3.054914e+00 [pb/GeV] or 3.054914e-01 [pb]
bin #27: 1.007721e+00 [pb/GeV] or 1.007721e-01 [pb]
bin #28: -nan [pb/GeV] or -nan [pb]

which are the correct values; bin 26 is unchanged, bin 27 has the average of two combined bins, and bin 28 must be cut away before generating the FK table.

I haven't checked the FK table generation, let me know if there are any problems - there might be because APPLgrid doesn't really combine the grids internally, only when convoluting.

@enocera
Copy link
Contributor

enocera commented Jan 24, 2021

@cschwan Thanks, it works. I've created a separate folder with the fixed data set, that I've renamed CDFZRAP_NEW consistently with the corresponding data implementation, see NNPDF/nnpdf#1068. See also NNPDF/apfelcomb#79

@enocera
Copy link
Contributor

enocera commented Apr 20, 2021

I'm closing this issue as it's been solved long ago.

@enocera enocera closed this as completed Apr 20, 2021
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

No branches or pull requests

2 participants