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

Bringup tt-torch models in forge #1314

Merged
merged 1 commit into from
Mar 4, 2025
Merged

Conversation

kamalrajkannan78
Copy link
Contributor

@kamalrajkannan78 kamalrajkannan78 commented Feb 25, 2025

Summary

  • This PR addresses Issue #1321 by porting models from tt-torch to tt-forge. The current compilation status of these models are available in the logs.

  • A list of skipped models along with the reasons for their exclusion can be found in Issue #1339. Any models that were missed in this PR will be included in PR #1337.

Note:

  • The following test cases are passing end-to-end (E2E), so a push marker & post processing steps has been added:
forge/test/models/pytorch/text/albert/test_albert.py::test_albert_sequence_classification_pytorch
forge/test/models/pytorch/text/perceiverio/test_perceiverio.py::test_perceiverio_masked_lm_pytorch
forge/test/models/pytorch/text/bert/test_bert.py::test_bert_question_answering_pytorch[phiyodr/bert-large-finetuned-squad2]

Logs:

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests604 ran478 passed126 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests604 ran477 passed126 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_dla.test_dla_pytorch[dla34]❌ failure

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests662 ran525 passed137 skipped0 failed
TestResult
No test annotations available

1 similar comment
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests662 ran525 passed137 skipped0 failed
TestResult
No test annotations available

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch from 66ff210 to 09430d9 Compare February 25, 2025 09:39
Copy link

TestsPassed ❌️SkippedFailed
TT-Forge-FE Tests0 ran0 passed0 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests662 ran525 passed137 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests604 ran478 passed126 skipped0 failed
TestResult
No test annotations available

Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests662 ran525 passed137 skipped0 failed
TestResult
No test annotations available

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch 2 times, most recently from 899912e to 823a81e Compare February 25, 2025 19:42
Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests4 ran0 passed0 skipped4 failed
TestResult
TT-Forge-FE Tests
pytest
test_bloom.forge.test.models.pytorch.text.bloom.test_bloom❌ failure
test_glpn_kitti.forge.test.models.pytorch.vision.glpn_kitti.test_glpn_kitti❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

3 similar comments
Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests4 ran0 passed0 skipped4 failed
TestResult
TT-Forge-FE Tests
pytest
test_bloom.forge.test.models.pytorch.text.bloom.test_bloom❌ failure
test_glpn_kitti.forge.test.models.pytorch.vision.glpn_kitti.test_glpn_kitti❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests4 ran0 passed0 skipped4 failed
TestResult
TT-Forge-FE Tests
pytest
test_bloom.forge.test.models.pytorch.text.bloom.test_bloom❌ failure
test_glpn_kitti.forge.test.models.pytorch.vision.glpn_kitti.test_glpn_kitti❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests4 ran0 passed0 skipped4 failed
TestResult
TT-Forge-FE Tests
pytest
test_bloom.forge.test.models.pytorch.text.bloom.test_bloom❌ failure
test_glpn_kitti.forge.test.models.pytorch.vision.glpn_kitti.test_glpn_kitti❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch from 823a81e to da83043 Compare February 26, 2025 09:11
Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests3 ran0 passed0 skipped3 failed
TestResult
TT-Forge-FE Tests
pytest
test_mlp_mixer.forge.test.models.pytorch.vision.mlp_mixer.test_mlp_mixer❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

3 similar comments
Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests3 ran0 passed0 skipped3 failed
TestResult
TT-Forge-FE Tests
pytest
test_mlp_mixer.forge.test.models.pytorch.vision.mlp_mixer.test_mlp_mixer❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests3 ran0 passed0 skipped3 failed
TestResult
TT-Forge-FE Tests
pytest
test_mlp_mixer.forge.test.models.pytorch.vision.mlp_mixer.test_mlp_mixer❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

Copy link

TestsPassedSkippedFailed ❌️
TT-Forge-FE Tests3 ran0 passed0 skipped3 failed
TestResult
TT-Forge-FE Tests
pytest
test_mlp_mixer.forge.test.models.pytorch.vision.mlp_mixer.test_mlp_mixer❌ failure
test_rmbg.forge.test.models.pytorch.vision.rmbg.test_rmbg❌ failure
test_yolos.forge.test.models.pytorch.vision.yolo.test_yolos❌ failure

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch 6 times, most recently from e8ebdde to 8b18209 Compare February 27, 2025 10:21
@kamalrajkannan78 kamalrajkannan78 marked this pull request as ready for review February 27, 2025 10:22
Copy link

github-actions bot commented Mar 1, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests677 ran540 passed137 skipped0 failed
TestResult
No test annotations available

1 similar comment
Copy link

github-actions bot commented Mar 1, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests677 ran540 passed137 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 1, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests618 ran489 passed129 skipped0 failed
TestResult
No test annotations available

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch from fb01ef7 to 5bd73b7 Compare March 3, 2025 10:01
Copy link

github-actions bot commented Mar 3, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests618 ran489 passed129 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 3, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests677 ran540 passed137 skipped0 failed
TestResult
No test annotations available

1 similar comment
Copy link

github-actions bot commented Mar 3, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests677 ran540 passed137 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 3, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests618 ran489 passed129 skipped0 failed
TestResult
No test annotations available

@meenakshiramanathan1 meenakshiramanathan1 self-requested a review March 3, 2025 10:50
@meenakshiramanathan1
Copy link
Contributor

LGTM

verify(inputs, framework_model, compiled_model)

# Inference
output = compiled_model(*inputs)
Copy link
Member

Choose a reason for hiding this comment

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

Take a look at this PR. When this merges into main, verify() would also return outputs of the forward pass.



def load_input():
url = "https://i.postimg.cc/ZKwLg2Gw/367-14.png"
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be nice to have one centralized folder e.g. data_inputs where we would store few images so all models would use the same inputs.

@kamalrajkannan78 kamalrajkannan78 force-pushed the kkannan/tt_torch_models_bringup branch from 5bd73b7 to 42c0352 Compare March 3, 2025 18:11
@kamalrajkannan78
Copy link
Contributor Author

@vkovinicTT. I created a separate issue(#1364) for this suggestion.
We can implement that in future as a separate PR.

@vkovinicTT
Copy link
Member

vkovinicTT commented Mar 4, 2025

@vkovinicTT. I created a separate issue(#1364) for this suggestion. We can implement that in future as a separate PR.

Awesome, thanks :)

Copy link
Member

@vkovinicTT vkovinicTT left a comment

Choose a reason for hiding this comment

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

LGTM, just waiting for the verify() change.

Copy link

github-actions bot commented Mar 4, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests626 ran486 passed140 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 4, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests684 ran544 passed140 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 4, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests626 ran486 passed140 skipped0 failed
TestResult
No test annotations available

Copy link

github-actions bot commented Mar 4, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests684 ran544 passed140 skipped0 failed
TestResult
No test annotations available

@kamalrajkannan78
Copy link
Contributor Author

LGTM, just waiting for the verify() change.

  • This verify() change will introduces the bunch of changes in all tests .To unblock this PR, can we handle verify() changes in separate PR ?

Copy link
Contributor

@ashokkumarkannan1 ashokkumarkannan1 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the PR @kamalrajkannan78

@vkovinicTT
Copy link
Member

LGTM, just waiting for the verify() change.

  • This verify() change will introduces the bunch of changes in all tests .To unblock this PR, can we handle verify() changes in separate PR ?

Yep, go ahead.

@vkovinicTT vkovinicTT merged commit 59941b6 into main Mar 4, 2025
10 checks passed
@vkovinicTT vkovinicTT deleted the kkannan/tt_torch_models_bringup branch March 4, 2025 13:21
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