Skip to content

Conversation

@felixhekhorn
Copy link
Contributor

No description provided.

@felixhekhorn felixhekhorn added the dependencies Pull requests that update a dependency file label Jun 12, 2025
@felixhekhorn
Copy link
Contributor Author

felixhekhorn commented Jun 12, 2025

@scarlehoff consider solving this one first, it might be easier (no numba, which is always a problem) - the funny thing is that we fail with a different package here: greenlet (triggered by pillow) and not scipy as in NNPDF/eko#470

@felixhekhorn
Copy link
Contributor Author

the problem here is: poetry wants to install greenlet==3.0.3 , but support for Python 3.13 is only starting from 3.1 - so poetry did not correctly resolve the dependencies, no idea on how to fix that ...

@scarlehoff
Copy link
Member

scarlehoff commented Jun 12, 2025

ah, good point. perhaps this is the issue in eko. still unsure how poetry lock didn't pick this up but in my local copy of eko i was simply skipping banana, I hacked this in two seconds just for the lock for the ci (which I know now is not enough)

@felixhekhorn
Copy link
Contributor Author

felixhekhorn commented Jun 12, 2025

The last commit is a stupid attempt just to see what happens now ... it did bump greenlet to 3.2.3 - but scipy is still at 1.13.1, which we know is a problem

@felixhekhorn
Copy link
Contributor Author

And indeed now, we fail with scipy: but actually the problem might be here:

| ../scipy/meson.build:163:9: ERROR: Dependency "OpenBLAS" not found, tried pkgconfig and cmake

@scarlehoff
Copy link
Member

That may explain why does it work locally 🧐

If I have time I'll have a go in a truly empty system. We should not be building scipy anyway.

@felixhekhorn
Copy link
Contributor Author

merge and release?

@scarlehoff
Copy link
Member

Yeah, let's do so. We'll see whether it creates a problem down the line.

@felixhekhorn felixhekhorn merged commit 1e602e1 into main Jun 13, 2025
5 checks passed
@felixhekhorn felixhekhorn deleted the relax_restrictions branch June 13, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants