whisperX icon indicating copy to clipboard operation
whisperX copied to clipboard

Update requirements.txt

Open jnuvvula9 opened this issue 1 year ago • 4 comments

faster-whisper was just updated a couple of hours ago

jnuvvula9 avatar Feb 22 '24 15:02 jnuvvula9

This PR also fixes this issue: https://github.com/m-bain/whisperX/issues/711

msfuko avatar Feb 23 '24 05:02 msfuko

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

See https://github.com/m-bain/whisperX/pull/710#issuecomment-1959400388

Terseus avatar Feb 23 '24 11:02 Terseus

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

davidmartinrius avatar Feb 29 '24 15:02 davidmartinrius

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.

deep-pipeline avatar Apr 14 '24 11:04 deep-pipeline