-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add positivity observables #124
Comments
Maybe this is the more suitable place then pineappl examples, even more if this will become a proper dataset (in the NNPDF sense) |
That was my train of thought also. I'd volunteer to take care of it, if you're busy with other projects. |
We're slowly completing most of the chores, but still I would not do it before ~10 days. If you can start, do it :) Were you thinking something about a new backend (like |
I was thinking about making it a new backend, which isn't too complicated given that we already have that script. |
Indeed, I just wanted to understand. At the end, a backend is not doing so much:
|
OK, thanks for your pointers, let me try to implement it! |
then it should be the runner version |
runner version is included always by default: (even though maybe we can even drop |
yes, I meant just not to leave it empty or something, but put the version twice (such that one can guess the generator, i.e. if they are the same the generator is built-in) |
No, why twice? It is redundant, and runner is runner, so leave it empty. Consider that there is no standard field for |
If you want to log the generator used let's add a separate field, and make it explicit (even though it might already be logged somewhere, but I'm not sure). |
Do we want a generator in this repository for the positivity observables? This would be basically https://github.com/N3PDF/pineappl/blob/master/examples/python/positivity.py.
The text was updated successfully, but these errors were encountered: