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

Uplift third_party/tt-metal to 0d4d99618a90640bd2a742535bb4f6c8ec6ba164 2025-01-14 #1770

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

vmilosevic
Copy link
Contributor

This PR uplifts the third_party/tt-metal to the 0d4d99618a90640bd2a742535bb4f6c8ec6ba164

@brataTT
Copy link
Contributor

brataTT commented Jan 14, 2025

Brings 9 tt-metal commits.

git log --oneline a6ec1c0a02c3b9dc55e769618639e0439ef9c06b..0d4d99618a90640bd2a742535bb4f6c8ec6ba164
0d4d99618a Finish porting OPs to compute_output_specs (#16695)
0c541b9884 #16621: Add barriers at end of cq_dispatch_slave.cpp on IERISC
5071e39bcc Update METALIUM_GUIDE.md (#16602)
20ee6812d6 Replace some instances of Tensor::get_shape with get_logical_shape (#16655)
4cfb561470 (tag: v0.54.0-rc23) #16587: Update UMD submodule commit for P150 compatibility
2e6915b805 Generate test executables per architecture (#16594)
ab4e9b5dc7 #0: Skip build-docker-image during post-commit code-analysis since the docker image is already built in a previous job (#16703)
d5dcbbe25c #0: Fixes from commit ae61802 (#16686)
ea6d5c0582 Tilize with val padding results in L1 cache OOM (#16633)

Passed tt-torch CI
Failed tt-forge-fe CI
This fail can be ignored. See why below.

The failed tests are:

2025-01-14T07:34:44.6469482Z FAILED forge/test/mlir/llama/tests/test_specific_ops_llama32.py::test_matmul[shapes6]
2025-01-14T07:34:44.6470201Z FAILED forge/test/mlir/mnist/training/test_training.py::test_mnist_training_with_grad_accumulation
2025-01-14T07:34:44.6470907Z FAILED forge/test/mlir/resnet/test_resnet_unique_ops.py::test_matmul_resnet[1-1000-2048]

Resolving them was discussed in last uplift PR
tt-forge was updated with commit

CI run did not include this commit.
Testing locally including this commit shows all 3 failed tests now pass.
Re-running to verify passed: https://github.com/tenstorrent/tt-forge-fe/actions/runs/12769180403

@brataTT brataTT merged commit 731de13 into main Jan 14, 2025
22 checks passed
@brataTT brataTT deleted the uplift branch January 14, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants