Skip to content

Conversation

@eshiryae
Copy link
Contributor

Description

E-190314

Checklist:

  • Tests have been updated or added to cover the new code.
  • This patch fully addresses the ticket.
  • I have made corresponding changes to the documentation.

@github-actions github-actions bot added the category: whisper Whisper pipeline label Nov 28, 2025
@github-actions github-actions bot added the category: GGUF GGUF file reader label Nov 28, 2025
@Wovchena Wovchena requested a review from Copilot December 3, 2025 05:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR switches the NPU Whisper pipeline to use the stateful pipeline by default instead of the static pipeline. The change involves reversing the default value of the STATIC_PIPELINE configuration option from true to false.

Key Changes:

  • Modified the default value of STATIC_PIPELINE from true to false in the WhisperPipeline implementation
  • Updated test configuration to explicitly set STATIC_PIPELINE to "YES" to maintain existing test behavior

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/cpp/src/whisper/pipeline.cpp Changed default value of STATIC_PIPELINE from true to false
tests/python_tests/test_whisper_pipeline_static.py Added explicit STATIC_PIPELINE configuration to preserve static pipeline testing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GGUF GGUF file reader category: whisper Whisper pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants