Closed Radonirinaunimi closed 2 years ago
F3 means xF3 right?
F3 means xF3 right?
Yes, indeed! I made this explicit in the description above.
Also these are for proton SFs? Do we also have grids for other nuclei?
Also these are for proton SFs? Do we also have grids for other nuclei?
These are only for proton SFs (sorry, I could have made this explicit in the naming), I will generate the other nuclei now.
This is now ready to be merged if anyone has time to review it. Otherwise I'd also be happy to merge it.
Since it performs a very simple task and doesn't really touch any existing code I would be happy to merge it without review.
Though from a code perspective i"m fine with this, I don't think we should publish the average in this way as mentioned on the slack channel.
Since it performs a very simple task and doesn't really touch any existing code I would be happy to merge it without review.
This is indeed something non-controversial but really important to get bug-free. So far, the results suggest that this is the case, and some of the functions anyway I took and modified from EKO. I am indeed in favor of merging this asap (without review).
Though from a code perspective i"m fine with this, I don't think we should publish the average in this way as mentioned on the slack channel.
I am not sure I understand what you mean about the average? I couldn't find a reference of this on the slack.
Nvm, forget I said anything. Feel free to merge
Add modules that dumps the NN predictions into LHAPDF-like grids.