-
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
Bump pineappl version #2129
Bump pineappl version #2129
Conversation
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.
If tests pass i think we can safely merge.
@Radonirinaunimi shouldn't we update also the |
Yes, I will do so once this NNPDF/pineappl#300 is resolved. I will have time to look into it more tomorrow (if not today). |
The problem is that the conda package for pineappl seems to not be working at the moment conda-forge/pineappl-feedstock#41 |
Are you happy with this @scarlehoff? |
Now that pineappl v0.8.1 is out we can update here. In principle, these changes are also introduced in #2110 but given that this is needed to make pineko work (NNPDF/pineko#181) it would be better to have this in master first.
Tests relying on conda are still failing because of NNPDF/pineappl#300.