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

Updating TT-MLIR commit reference #30

Merged
merged 10 commits into from
Aug 2, 2024

Conversation

sdjordjevicTT
Copy link
Contributor

@sdjordjevicTT sdjordjevicTT commented Jul 30, 2024

Updating TT-MLIR commit reference to include the fix for Softmax op that can now accept negative dimension argument.

@sdjordjevicTT sdjordjevicTT requested a review from pilkicTT July 30, 2024 14:30
Copy link
Contributor

@pilkicTT pilkicTT left a comment

Choose a reason for hiding this comment

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

Did you run the existing test, just to confirm that everything works?

tt-mjudge and others added 9 commits August 2, 2024 08:38
(cherry picked from commit d8cb16db58c63d101bda5137b7a83ca6954d761f)
(cherry picked from commit 116dd3b147fd6f31968febc2d24c4c92c0b2a533)
(cherry picked from commit 48e64b159e17114ec083102170f39f58a524b9e0)
(cherry picked from commit 179d7adc3e822056f0a0cc242356963b0cd3ce1c)
…_models and nightly test

and configuration file

(cherry picked from commit 26fb354ffb75f37e8394f79ff2566e91287d14a5)
(cherry picked from commit 5fb09ba4d26011b302e60297b83352031ccfd7c0)
(cherry picked from commit 88b29efb09cdf658b9a05ce977f99285bd242205)
(cherry picked from commit 20cc1350f85a9c0f2fbdf820973b53477ebeed0a)
(cherry picked from commit 1283b4bd3d5e7ed53f75d20b3b165bafe004beb6)
@sdjordjevicTT sdjordjevicTT force-pushed the sdjordjevic/uplifting_tt_mlir_commit branch 2 times, most recently from 81af1a5 to d9d9e0f Compare August 2, 2024 08:44
@sdjordjevicTT sdjordjevicTT force-pushed the sdjordjevic/uplifting_tt_mlir_commit branch from d9d9e0f to 26d5b7f Compare August 2, 2024 16:25
@sdjordjevicTT
Copy link
Contributor Author

Yes, I run this set of tests:
pybuda/test/test_api.py
pybuda/test/mlir/test_ops.py::test_add
pybuda/test/mlir/test_ops.py::test_multiply

They are passing.

@sdjordjevicTT sdjordjevicTT merged commit 926a2b1 into main Aug 2, 2024
1 check failed
@sdjordjevicTT sdjordjevicTT deleted the sdjordjevic/uplifting_tt_mlir_commit branch August 5, 2024 12:30
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.

9 participants