Skip to content
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

Python vesion restrict should be added. #2

Open
zzkluck opened this issue May 5, 2024 · 0 comments
Open

Python vesion restrict should be added. #2

zzkluck opened this issue May 5, 2024 · 0 comments

Comments

@zzkluck
Copy link

zzkluck commented May 5, 2024

When I built the environment dependencies, I got the following error:

Collecting scikit-learn==1.1.2 (from -r .\requirements.txt (line 2))
...
...
      AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?

I looked it up and found that I was using a very new version of python, python3.12 is not compatible with scikit-learn==1.1.2, which need python version less than 3.11. So I realized that the python version is not arbitrary. Further, I found that torch==1.10.1 limits the highest python version to py3.8, see this link.

I believe it would be helpful to others if you could provide the version of python you use directly in the readme.

But is the torch version of the qualification necessary? In my very personal experience, torch's compatibility is not bad, probably torch2 will work just as well? You know, it is sometimes not easy to install a lower version of the torch because of issues such as GPU drivers.

Finally, nltk and pillow are also needed, but not in requirements.txt.

Thank you for your effort!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant