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

Update requirements.txt #657

Merged
merged 1 commit into from
Jun 29, 2024
Merged

Update requirements.txt #657

merged 1 commit into from
Jun 29, 2024

Conversation

77degrees
Copy link
Contributor

numpy got updated to 2.0 and py3landid added support for numpy 2.0. Try the below, and it should work; it did for me.

pip install py3landid==0.2.2
pip install numpy==1.23.5

numpy got updated to 2.0 and py3landid added support for numpy 2.0. Try the below, and it should work; it did for me.

pip install py3landid==0.2.2
pip install numpy==1.23.5
@zyddnys zyddnys merged commit 7caba85 into zyddnys:main Jun 29, 2024
@zyddnys
Copy link
Owner

zyddnys commented Jun 29, 2024

why did you remove pydensecrf and langdetect?

jokester added a commit to moeflow-com/manga-image-translator that referenced this pull request Jul 3, 2024
a recent version is necessary after zyddnys#645

but it got incorrected downgraded in zyddnys#657
jokester added a commit to moeflow-com/manga-image-translator that referenced this pull request Jul 3, 2024
a recent version is necessary after zyddnys#645

but it got incorrectly downgraded in zyddnys#657
@jokester
Copy link
Contributor

jokester commented Jul 3, 2024

The numpy change was valid but pydencerf and openai should not really be touched.

The old openai==0.28.0 breaks after #645

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.

3 participants