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

Enable ConvMnist and Mnist integration and performance tests. #16965

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

Conversation

kkeerthana0573
Copy link
Contributor

Ticket

Link to Github Issue

Problem description

Re-enable ConvMnist and Mnist integration and performance tests.

What's changed

ConvMnist and Mnist integration test failed on all cards are now fixed with the latest main.

Checklist

  • Post commit CI passes
  • Blackhole Post commit (if applicable)
  • Model regression CI testing passes (if applicable)
  • Device performance regression CI testing passes (if applicable)
  • (For models and ops writers) Full new models tests passes
  • New/Existing tests provide coverage for changes

@kkeerthana0573 kkeerthana0573 changed the title #0: Test Mnist and ConvMnist integration tests Enable ConvMnist and Mnist integration and performance tests. Jan 22, 2025
@saichandax
Copy link
Contributor

@Sudharsan-V , @kkeerthana0573
Please post passing CI links in the description when available.

@Sudharsan-V
Copy link
Contributor

Both the Nightly and Device perf CIs are successfully passing for MNIST and ConvNet MNIST.

Nightly CI Nightly CI and ttnn tests.
Device Perf Mnist, convnet_mnist
Note: Device perf CI is currently unstable. Therefore, the links for mnist and convnet_mnist have been highlighted and shared shared above

@kkeerthana0573 kkeerthana0573 force-pushed the keerthana/ttnn_convmnist_check branch from 60234f5 to 966dc44 Compare January 23, 2025 06:57
@tt-rkim
Copy link
Collaborator

tt-rkim commented Jan 27, 2025

Approving to unblock
I would recommend trying now, as device perf should be stable

@kkeerthana0573 kkeerthana0573 force-pushed the keerthana/ttnn_convmnist_check branch 3 times, most recently from 0716609 to 3646be3 Compare January 29, 2025 12:26
@kkeerthana0573 kkeerthana0573 force-pushed the keerthana/ttnn_convmnist_check branch from 3646be3 to 5b477b7 Compare February 3, 2025 06:22
@kkeerthana0573
Copy link
Contributor Author

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.

5 participants