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

Potential fix for #90, #91 #92

Merged
merged 5 commits into from
Oct 4, 2024
Merged

Potential fix for #90, #91 #92

merged 5 commits into from
Oct 4, 2024

Conversation

KoljaB
Copy link

@KoljaB KoljaB commented Oct 3, 2024

Fixing versions of dependencies in the pyproject.toml.

Probably some version restrictions could be lowered.

dependencies = [
#...
"cython==3.0.11", # Added exact version
"spacy[ja]==3.7.5", # Added exact version
"soxr==0.4.0", # Added exact version
"scikit-learn==1.5.1", # Added exact version
"contourpy==1.2.1", # Added exact version
]

Fixes #91

Copy link
Member

@eginhard eginhard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for looking into this! I tested a bit locally and the minimal required change for it to work again should be setting "spacy[ja]>=3,<3.8",, so I just simplified this PR accordingly (also keeping the Cython update). I'll prepare a release with this.

@eginhard eginhard merged commit f75d095 into idiap:dev Oct 4, 2024
48 checks passed
eginhard added a commit that referenced this pull request Oct 4, 2024
* Update pyproject.toml

* Update pyproject.toml

* Update pyproject.toml

* Update pyproject.toml

* build: simplify requirement restrictions

---------

Co-authored-by: Enno Hermann <[email protected]>
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

Successfully merging this pull request may close these issues.

[Bug] Install fails with: No module named 'distutils.msvccompiler'
2 participants