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

remove ND failing big shape in transpose failures that is already tracked and disabled in test_transpose_2d #17145

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

sjameelTT
Copy link
Contributor

@sjameelTT sjameelTT commented Jan 27, 2025

Ticket

#17144

Problem description

This shape fails N-D on main, starting about a month and a half ago

What's changed

Since this shape is tested earlier in the suite, I'm removing it.

Checklist

Copy link
Collaborator

@tt-rkim tt-rkim left a comment

Choose a reason for hiding this comment

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

Will approve to unblock, but I'd recommend placing a comment with the new issue number rather than deleting the case

@sjameelTT sjameelTT merged commit e0ed585 into main Jan 27, 2025
189 of 191 checks passed
@sjameelTT sjameelTT deleted the sjameel/disable_big_transpose branch January 27, 2025 18:31
williamlyTT pushed a commit that referenced this pull request Jan 30, 2025
…cked and disabled in test_transpose_2d (#17145)

### Ticket
#17144

### Problem description
This shape fails N-D on main, starting about a month and a half ago

### What's changed
Since this shape is tested earlier in the suite, I'm removing it.

### Checklist
- [ ] Post commit CI passes
https://github.com/tenstorrent/tt-metal/actions/runs/12993059516
- [ ] 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](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
yieldthought pushed a commit that referenced this pull request Jan 31, 2025
…cked and disabled in test_transpose_2d (#17145)

### Ticket
#17144

### Problem description
This shape fails N-D on main, starting about a month and a half ago

### What's changed
Since this shape is tested earlier in the suite, I'm removing it.

### Checklist
- [ ] Post commit CI passes
https://github.com/tenstorrent/tt-metal/actions/runs/12993059516
- [ ] 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](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml)
tests passes
- [ ] New/Existing tests provide coverage for changes
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.

2 participants