-
Notifications
You must be signed in to change notification settings - Fork 584
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
[BUG] installation of version 0.14.8 fails #1052
Comments
Cython released a major release (3.0) last week. This changed some major things, and is almost certainly the cause of this error The solution would be to limit Cython to disallow version 3.0, but I can't find where in the install process it specifies the Cython version :/ |
The installation fails with the same error with cythons: Is there a workaround, the produciton code fails to install environments? |
Maybe there was an update of pomegranate PYPI packages? Because if I build wheel from an old vesion, it installs correctly. |
Installing a specific cython version locally before build does not - unfortunately - mean that Pomegranate will use that version to build... |
None, I am unable to install ~=.28.0, because these versions incompatiable with python 3.10.6. |
Sorry, only now do I understand what's going on The Cython was required from https://github.com/jmschrei/pomegranate/blob/272039bf3fa208a9ec0f67b53f03cea280abcc1b/setup.py (in setup_requires section) So in that file you'd need to limit the Cython version to now be 3.0... but all those versions are already deprecated :( |
Can you explain please, I succesfully installed cython 0.29.35, so pomegranate during installation used this version. However the compilation still fails. |
When older versions of pomegranate install themselves, they download their own set of build dependencies - as shown above in the setup_requires section - for their installation process only. The old setup.py specified to download "cython >= 0.22.1" - but with the way that pip version resolution works, this means that it will try to take the latest version of Cython, and thus (since last week) would download Cython 3.0 which fails to compile the library :/ |
So the workaround is to download the wheel and patch the setup.py and then install from local wheel? |
I can't, only the repo owner can upload packages to pypi :/ |
Can you download a version of Cython around 0.22 first and then try to pip install? |
Will do 👍🏿 |
It would not let me |
I was able to get an install working by pinning the cython version in build-system in pyproject.toml to enable |
Is there a way to re-release tag 0.14.8 with these patches? |
For now what I did in my pyproject.toml is: pomegranate = { git = "https://github.com/yairm210/pomegranate", branch = "0.14.8" } and it works :) So as a temporary solution you can use that |
Stuck here too. Strange thing is I can get it to work on my local windows machine (Python==3.9.12 && Cython==3.0.0) but not any other machine I have tried (all linux). They fail with this exact error. I use v0.14.8 because it's plays nice with some tools I use over Bayesian Networks. Also it's more feature rich than v1.0.0 for now. Would be great to keep this version alive until 1.0.0 really takes over. I love the torch implementation. |
I forked v0.14.8 and added the changes suggested by @kkchau. If you could confirm that this resolves the issues you're facing, I'll release it on PyPI. https://github.com/jmschrei/pomegranate/tree/v0.14.9 |
@jmschrei Confirmed that that branch should work!
|
Hi @jmschrei, wanted to check in and see if you had a timeline for releasing 0.14.9? |
@kkchau I just got back from some travel. Can you try |
Ah sorry, yeah it works now! Thanks for the update |
No problem, thanks for checking! @tyrrellgit does 0.14.9 work for you as well? |
Describe the bug
Installation of version 0.14.8 fails with the following exception:
It happened on Jul. 19. Before that date the installation worked.
To Reproduce
Response time
Versions 0.14.7, 0.14.6 also fail.
The text was updated successfully, but these errors were encountered: