tests : remove gh label test-whisper-cli-tiny-en #2988
Merged
Conversation
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
This commit removes test-whisper-cli-tiny-en from the gh label. The motivation for this change is that until recently the tests were disabled. But now that they are enabled some of the tests, specifically the ci jobs that use sanatizers (e.g. thread-sanitizer) take a long time to run as they are instrumented. Some of these jobs also have matricies which means that there are multiple jobs are created that all run these tests. The suggestion here is to limit the number of tests that are run in the ci jobs so cut down the CI build time.
ggerganov
approved these changes
Apr 2, 2025
bygreencn
added a commit
to bygreencn/whisper.cpp
that referenced
this pull request
Jun 29, 2025
* tag 'v1.7.5': (256 commits) release : v1.7.5 bench : update numbers [no ci] (ggml-org#2993) sync : ggml get_rows and dup optimization (llama/12671) opencl : fix memory allocation size (llama/12649) metal : use F32 prec in FA kernels (llama/12688) Fix clang warning in gguf_check_reserved_keys (llama/12686) vulkan: fix build when glslc doesn't support coopmat (llama/12683) SYCL: Rename oneMKL to oneMath (llama/12192) SYCL: switch to SYCL namespace (llama/12674) ggml : faster ssm scan (llama/10558) Vulkan: Add DP4A MMQ and Q8_1 quantization shader (llama/12135) cmake : fix whitespace (llama/0) tests : remove gh label test-whisper-cli-tiny-en (ggml-org#2988) examples : clarify Core ML encoder model usage [no ci] (ggml-org#2987) ci : remove intermediate build on push to master (ggml-org#2986) whisper.objc : fix typo in README.md [no ci] (ggml-org#2985) coreml: fix Whisper to CoreML conversion by disabling SDPA [no ci] (ggml-org#2979) ci : add coreml job that converts base.en to coreml [no ci] (ggml-org#2981) tests : re-enable tests [no ci] (ggml-org#2977) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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 removes
test-whisper-cli-tiny-enfrom theghlabel.The motivation for this change is that until recently the tests were disabled. But now that they are enabled some of the tests, specifically the ci jobs that use sanatizers (e.g. thread-sanitizer) take a long time to run as they are instrumented.
Some of these jobs also have matrices which means that there are multiple jobs are created that all run these tests. The suggestion here is to limit the number of tests that are run in the ci jobs so cut down the CI build time.