Carleslc / AudioToText

Transcribe and translate audio to text using Whisper and DeepL.
https://carleslc.me/AudioToText
264 stars 33 forks source link

Update Whisper Packge to 20231117 #6

Closed johnnylai closed 9 months ago

johnnylai commented 9 months ago

After I update the Whisper package the dependency error is resolved. Which solve the issue https://github.com/Carleslc/AudioToText/issues/5

Carleslc commented 9 months ago

Thank you for the PR and noticing the issue. Version tag should be v20231117, the initial v is missing:

Collecting git+https://github.com/openai/whisper.git@20231117
  Cloning https://github.com/openai/whisper.git (to revision 20231117) to /tmp/pip-req-build-54x8uf9z
  Running command git clone --filter=blob:none --quiet https://github.com/openai/whisper.git /tmp/pip-req-build-54x8uf9z
  WARNING: Did not find branch or tag '20231117', assuming revision or ref.
  Running command git checkout -q 20231117
  error: pathspec '20231117' did not match any file(s) known to git
  error: subprocess-exited-with-error

  × git checkout -q 20231117 did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.

However, with that there were still dependency issues:

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
llmx 0.0.15a0 requires cohere, which is not installed.

So I also updated other packages to fix this dependency issue, also updated openai to 0.28.

For now this PR solves the issue. I could also update openai to the most recent 1.8.0 but the API changed so I'd have to migrate the code.

Carleslc commented 9 months ago

openai library now is also updated to the most recent version 1.9.0

johnnylai commented 8 months ago

Thank you!