-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
NeMo installation fails due to outdated abandoned dependency on youtokentome #6854
Comments
Cython is a requirement of NeMo, so that must always be installed before other libraries. However, we have installed NeMo on python 3.9 for some time without issue using pip. Though the cython itself has been installed a long time ago so that might be an issue. Our NMT models depend on YTTM, so worst comes to worst we'll have to find an alternative and deprecate these models. But first we'll take a look at possible resolutions |
This issue is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
This issue was closed because it has been inactive for 7 days since being marked as stale. |
Is there a workaround? I can't install nemo_toolkit in my venv under Windows 10. |
This issue is very real, because even though there is a workaround when installing individual libraries using pip or poetry, trying to install from The easiest workaround seems to be to merge the PR @svandiekendialpad mentioned in a fork of the original repo, and then install that project before installing I have taken this approach, creating the fork @https://github.com/LahiLuk/YouTokenToMe. Leaving it here in case it can help someone as a workaround. |
This is still an issue on my end, Cython is not installed by default on my EC2 servers, and having to preinstall a package for production adds unnecessary complexity. This is frustrating since the workaround is very straightforward and @LahiLuk already provided the PR. Since the project is abandoned, another workaround would be to specify |
@LahiLuk thanks for your fork. It mentions that it has been tested with Python 3.10. I seem to be able to use |
Hi @svandiekendialpad, sorry for the late reply. If this is still a problem, could you please open an issue in https://github.com/LahiLuk/YouTokenToMe/issues? Then I can investigate further. Thanks! |
@titu1994 - can you please take a look at this again? The latest docs say NeMo is dependent on Python 3.10 or higher, but the installation fails with python>3.9. https://docs.nvidia.com/deeplearning/nemo/user-guide/docs/en/main/starthere/intro.html#installation |
Fails how? The CI is running Nemo daily on 3.10.12 |
We're basically removing yttm entirely, in #8322 |
I get a ModuleNotFound error for Cython, even though it's installed. Doing |
Spend almost a day trying to figure out what is going on with cython moduleNotFound error. Thanks Nvidia for still not fixing this. |
Now I get, RuntimeError: pybind11 install failed. Wow, never ends. |
^ Yup, I received the same error 😕 |
Okay, these steps actually worked for me! 🎉
|
I get this error when running those steps above `Collecting megatron-core==0.4.0 (from nemo_toolkit[all]) × Getting requirements to build wheel did not run successfully. |
Can you try pip install --upgrade nemo_toolkit[all] to make sure you're trying to install nemo 1.23? |
Running pip3 install --upgrade --no-cache-dir "nemo_toolkit[all]" on python3.10.12, Cython is installed, but getting
|
Also getting this. |
Describe the bug
NeMo installation fails with Python >=3.9 because
youtokentome
does not properly specify it's build dependency oncython
. See this issue in their repo.. There is also a PR on that repo to fix that but the author seems to have abandoned the project, its last release was in 2020.Steps/Code to reproduce bug
Trying to install
nemo_toolkit
using Python 3.9 or above.Expected behavior
Installation should succeed.
Environment overview (please complete the following information)
poetry add nemo_toolkit[asr]
Environment details
If NVIDIA docker image is used you don't need to specify these.
Otherwise, please provide:
Additional context
This is particularly annoying given that the whole package depends on
youtokentome
(seerequirements_common.txt
). So even if users like me only care about the ASR part,youtokentome
must be installed.The text was updated successfully, but these errors were encountered: