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

Bump pineappl version #2129

Merged
merged 4 commits into from
Jul 23, 2024
Merged

Bump pineappl version #2129

merged 4 commits into from
Jul 23, 2024

Conversation

Radonirinaunimi
Copy link
Member

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.

Copy link
Contributor

@giacomomagni giacomomagni left a 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.

@giacomomagni
Copy link
Contributor

@Radonirinaunimi shouldn't we update also the conda-recipe to make all the test passing?

@Radonirinaunimi
Copy link
Member Author

@Radonirinaunimi shouldn't we update also the conda-recipe to make all the test passing?

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).

@scarlehoff
Copy link
Member

The problem is that the conda package for pineappl seems to not be working at the moment conda-forge/pineappl-feedstock#41

@Radonirinaunimi
Copy link
Member Author

Are you happy with this @scarlehoff?

pyproject.toml Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants