Skip to content

Update to 3.14 - #257

Draft
jekoorn wants to merge 3 commits into
mainfrom
py314
Draft

jekoorn wants to merge 3 commits into
mainfrom
py314

Conversation

@jekoorn

@jekoorn jekoorn commented Oct 27, 2026 •

Copy link
Copy Markdown

I think in the end it boils down to regenerating the lockfile.
Edit: and the workflows.

Do we want to do something similar as in https://github.com/NNPDF/eko/pull/528/changes
with installing python in the bench, instead of loading an image?

@jekoorn
jekoorn marked this pull request as draft October 28, 2026 00:13
@felixhekhorn

Copy link
Copy Markdown
Contributor

Do we want to do something similar as in https://github.com/NNPDF/eko/pull/528/changes
with installing python in the bench, instead of loading an image?

in principle yes (to avoid complicated image generation), but as you can see we are getting in trouble with LHAPDF - however, maybe we can get around that thanks to NeoPDF? @Radonirinaunimi ? i.e. we would need to rewrite the benchmarks to use NeoPDF, which should be easy?

(Locally LHAPDF works for you, right?)

@Radonirinaunimi

Copy link
Copy Markdown
Member

in principle yes (to avoid complicated image generation), but as you can see we are getting in trouble with LHAPDF - however, maybe we can get around that thanks to NeoPDF? @Radonirinaunimi ? i.e. we would need to rewrite the benchmarks to use NeoPDF, which should be easy?

Yes 😃 I'd actually advocate to replace all the containers and custom-build of LHAPDF in the CI with NeoPDF 🙈

Making those changes should be as easy as:

from neopdf.pdf import PDF as lhapdf

and the rest should be exactly the same.

@jekoorn

jekoorn commented Oct 28, 2026 •

Copy link
Copy Markdown
Author

(Locally LHAPDF works for you, right?)

Locally LHAPDF is giving me issues, too. Unsure yet what exactly is going wrong. It also has trouble finding the pineko.toml (I added it)
Let's see what happens when I use neoPDF

This branch has not been deployed

No deployments
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.

3 participants