Skip to content

Commit

Permalink
Fix Whisper.cpp on Windows (#422)
Browse files Browse the repository at this point in the history
  • Loading branch information
chidiwilliams authored Apr 29, 2023
1 parent cb5ad74 commit 66bd9a1
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .coveragerc
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ omit =
directory = coverage/html

[report]
fail_under = 68
fail_under = 76
12 changes: 11 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,12 @@ jobs:
sudo apt install -y libpulse-mainloop-glib0 libegl1-mesa-dev libgstreamer-plugins-base1.0-dev libgstreamer1.0-dev libportaudio2
fi
if [ "$RUNNER_OS" == "Windows" ]; then
curl -OL "https://github.com/ggerganov/whisper.cpp/releases/download/v1.3.0/whisper-bin-x64.zip"
unzip whisper-bin-x64.zip
mv main.exe whisper_cpp.exe
fi
poetry run make test
shell: bash

Expand Down Expand Up @@ -146,7 +152,11 @@ jobs:
poetry run make bundle_mac
elif [ "$RUNNER_OS" == "Windows" ]; then
curl -OL "https://github.com/ggerganov/whisper.cpp/releases/download/v1.3.0/whisper-bin-x64.zip"
unzip whisper-bin-x64.zip
mv main.exe whisper_cpp.exe
poetry run make bundle_windows
elif [ "$RUNNER_OS" == "Linux" ]; then
Expand Down
1 change: 0 additions & 1 deletion tests/transcriber_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ def test_should_transcribe(self, qtbot):
assert 'Bienvenue dans Passe' in text


@pytest.mark.skipif(platform.system() == 'Windows', reason='whisper_cpp not printing segments on Windows')
class TestWhisperCppFileTranscriber:
@pytest.mark.parametrize(
'word_level_timings,expected_segments',
Expand Down

0 comments on commit 66bd9a1

Please sign in to comment.