-
Notifications
You must be signed in to change notification settings - Fork 122
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
Update cython version #88
Conversation
@@ -1,5 +1,5 @@ | |||
numpy | |||
matplotlib | |||
argparse | |||
cython | |||
cython<=0.29.36 |
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.
Do we require cython as runtime requirement?
It has nothing to do with Cython version. I've confirmed that. |
Hey @JeremyCCHsu, I see installation errors with Cython 3.0.0. How did you confirm that? In my case restricting cython version works okay r9y9/pysptk#96. |
Hi @r9y9 . This comment has the hint-- What I have problem with PYPI is Travis CI-- it failed to deploy due to some Anyway, installing from current master is working. conda create -yn testpw0720 python=3.10
pip cache purge
git clone https://github.com/JeremyCCHsu/Python-Wrapper-for-World-Vocoder.git
cd Python-Wrapper-for-World-Vocoder
git submodule update --init
pip install . For testing: pip install matplotlib soundfile
cd demo
python demo.py The culprit isn't Cython: import pyworld
import cython
>>> cython.__version__
'3.0.0' |
My previous conjecture about the package order in
#error Do not use this file, it is the result of a failed Cython compilation. Anyways, the above are just some of my observations FYI, @r9y9. |
There is an error raising when installing pyworld through pip since July 17th, the error occurs because of the newly released cython, restricting the cython version to <=0.29.36 can help solve the error. Local test succeeded.