Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#8246: functional_whisper CG test_demo #8493

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kkeerthana0573
Copy link
Contributor

This PR contains test_demo for functional_whisper Conditional Generation.

@kkeerthana0573 kkeerthana0573 force-pushed the keerthana/functional_whisper_test_demo branch 3 times, most recently from 41560e1 to f37a389 Compare May 21, 2024 08:14
@tt-rkim
Copy link
Collaborator

tt-rkim commented May 22, 2024

@kkeerthana0573
Pls review the following errors in nightly pipeline:

FAILED tests/ttnn/integration_tests/whisper/test_ttnn_functional_whisper.py::test_encoder[sequence_length=3000-feature_size=80-batch_size=1-model_name=openai/whisper-base-ttnn_model=models.experimental.functional_whisper.tt.ttnn_functional_whisper] - TypeError: encoder() missing 1 required keyword-only argument: 'device'
FAILED tests/ttnn/integration_tests/whisper/test_ttnn_functional_whisper.py::test_ttnn_whisper[ttnn_model=models.experimental.functional_whisper.tt.ttnn_functional_whisper] - TypeError: whisper() missing 1 required keyword-only argument: 'device'
FAILED tests/ttnn/integration_tests/whisper/test_ttnn_optimized_functional_whisper.py::test_encoder[sequence_length=3000-feature_size=80-batch_size=1-model_name=openai/whisper-base-ttnn_model=models.experimental.functional_whisper.tt.ttnn_optimized_functional_whisper] - TypeError: encoder() missing 1 required keyword-only argument: 'device'
FAILED tests/ttnn/integration_tests/whisper/test_ttnn_optimized_functional_whisper.py::test_ttnn_whisper[ttnn_model=models.experimental.functional_whisper.tt.ttnn_optimized_functional_whisper] - TypeError: whisper() missing 1 required keyword-only argument: 'device'

Thank you for running. Don't forget to encourage your fellow MCW team members to continue such practices

@kkeerthana0573 kkeerthana0573 force-pushed the keerthana/functional_whisper_test_demo branch 2 times, most recently from 0a5f494 to 0948b04 Compare May 22, 2024 14:57
@kkeerthana0573
Copy link
Contributor Author

@tt-rkim,
Both the tests are updated.
All whisper integration tests are updated in #8339. To avoid further conflicts, I shall merge this PR after #8339 is merged.
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants