ci : use dynamic libopenblas.dll for window-blas#3177
Merged
Conversation
This commit updates the windows-blas job to use the dynamic (can load different kernels depending of the CPU arch) libopenblas.dll instead of the "static" openblas.dll that get installed by vcpgk. The motivation for this change is that there have been reports of performance drops in later version specifically related to blas. Please see the links below for more details. Resolves: ggml-org#3166 Refs: ggml-org#2666 (comment)
Member
Author
|
To tests this, download whisper-blas-bin-x64.zip.zip and then extract it: > mkdir temp
> tar xvf .\whisper-blas-bin-x64.zip.zip
> tar xvf .\whisper-blas-bin-x64.zip
> cd .\Release\
> ls
Directory: C:\win-temp-dir\Release
Mode LastWriteTime Length Name
---- ------------- ------ ----
-a---- 5/21/2025 6:36 AM 27136 bench.exe
-a---- 5/21/2025 6:36 AM 27136 command.exe
-a---- 5/21/2025 6:36 AM 494080 ggml-base.dll
-a---- 5/21/2025 6:36 AM 54272 ggml-blas.dll
-a---- 5/21/2025 6:37 AM 456192 ggml-cpu.dll
-a---- 5/21/2025 6:37 AM 69120 ggml.dll
-a---- 1/12/2025 4:34 PM 51126150 libopenblas.dll
-a---- 5/21/2025 6:37 AM 184320 lsp.exe
-a---- 5/21/2025 6:37 AM 27136 main.exe
-a---- 5/21/2025 6:37 AM 108032 quantize.exe
-a---- 11/2/2023 5:05 PM 2500096 SDL2.dll
-a---- 5/21/2025 6:37 AM 27136 stream.exe
-a---- 5/21/2025 6:37 AM 363008 test-vad-full.exe
-a---- 5/21/2025 6:37 AM 363008 test-vad.exe
-a---- 5/21/2025 6:37 AM 370688 vad-speech-segments.exe
-a---- 5/21/2025 6:37 AM 165888 wchess.exe
-a---- 5/21/2025 6:37 AM 20480 whisper-bench.exe
-a---- 5/21/2025 6:37 AM 480768 whisper-cli.exe
-a---- 5/21/2025 6:37 AM 167936 whisper-command.exe
-a---- 5/21/2025 6:37 AM 713216 whisper-server.exe
-a---- 5/21/2025 6:38 AM 388096 whisper-stream.exe
-a---- 5/21/2025 6:39 AM 1529344 whisper-talk-llama.exe
-a---- 5/21/2025 6:37 AM 492032 whisper.dllNotice that we are now including
|
Member
Author
|
I'll go over the other upload steps in the next few days and make sure they are consistent and don't have the double .zip file extension as well. |
ggerganov
approved these changes
May 22, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit updates the windows-blas job to use the dynamic (can load different kernels depending of the CPU arch) libopenblas.dll instead of the "static" openblas.dll that get installed by vcpgk.
The motivation for this change is that there have been reports of performance drops in later version specifically related to blas. Please see the links below for more details.
Resolves: #3166
Refs: #2666 (comment)