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 #713

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update requirements.txt #713

wants to merge 1 commit into from

Conversation

jnuvvula9
Copy link

faster-whisper was just updated a couple of hours ago

@msfuko
Copy link

msfuko commented Feb 23, 2024

This PR also fixes this issue: #711

@Terseus
Copy link

Terseus commented Feb 23, 2024

@jnuvvula9 Please, can you change the requirement to use pypi (faster-whisper >= 1.0.0) instead of installing from Github?

See #710 (comment)

@davidmartinrius
Copy link
Contributor

This update won't work. I already tested it.

@deep-pipeline
Copy link

Just visiting the repo and looking at the 29 PRs and thinking that some are duplicates or overlapping PRs which have been overtaken and could be dropped with a bit of sanity checking to close off loose ends.

@davidmartinrius has this PR ( #713 ) been overtaken by any further updates and/or can you clarify if issue #711 is still live issue or not (i.e. can it closed - I've nudged the original poster).

If so we could maybe suggest to repo owner that this PR request is closed or else a suitable fix is incorporated if one is needed? Thanks.

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.

5 participants