#0: Remove hardcoded grid width in all_gather and skip test_sharded_matmul test when the device grid size is too small #16315
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.
Ticket
Link to Github Issue
Problem description
All gather had hardcoded device width of 8 but should derive from device grid size.
Sharded MM unit test did not check if the shard config actually fits on the device grid.
What's changed
Change all gather to pull width from device.
Skip sharded MM test if device grid is too small.
Checklist
T3K unit: https://github.com/tenstorrent/tt-metal/actions/runs/12501718468
T3K freq: https://github.com/tenstorrent/tt-metal/actions/runs/12506425069
T3K nightly: https://github.com/tenstorrent/tt-metal/actions/runs/12506426406/job/34894919296
T3K perf: https://github.com/tenstorrent/tt-metal/actions/runs/12506429619
TG freq: https://github.com/tenstorrent/tt-metal/actions/runs/12506431709
TG nightly: https://github.com/tenstorrent/tt-metal/actions/runs/12506432943