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.

Sign up for GitHub

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

Jump to bottom

Add TVM decomposition for masked_scatter op #1188

Merged
merged 1 commit into from
Feb 26, 2025
Merged

Add TVM decomposition for masked_scatter op #1188

merged 1 commit into from
Feb 26, 2025

Conversation

ashokkumarkannan1
Copy link
Contributor

@ashokkumarkannan1 ashokkumarkannan1 commented Feb 6, 2025 •
edited
Loading

Ticket

Fix #1187

What's changed

@ashokkumarkannan1 ashokkumarkannan1 mentioned this pull request Feb 6, 2025
Merged
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests510 ran451 passed59 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests568 ran473 passed79 skipped16 failed
TestResult
TT-Forge-FE Tests
pytest
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape3]❌ failure

1 similar comment
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests568 ran473 passed79 skipped16 failed
TestResult
TT-Forge-FE Tests
pytest
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape0-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape1-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape2-indices_shape3]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape0]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape1]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape2]❌ failure
test_tm.test_adv_index_embedding_decompostion[input_tensor_shape3-indices_shape3]❌ failure

@nvukobratTT
Copy link
Contributor

nvukobratTT commented Feb 6, 2025 •
edited
Loading

@ashokkumarkannan1 seems like a few of tests are failing. Can we fix those before pushing for review? Until then, it can probably be a Draft PR? :))

@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch 4 times, most recently from 05d38a5 to 161f799 Compare February 7, 2025 13:53
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests648 ran508 passed139 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_scatter_ops.test_masked_scatter[2d_masked_scatter]❌ failure

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests590 ran465 passed124 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_scatter_ops.test_masked_scatter[1d_masked_scatter]❌ failure

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests648 ran508 passed139 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_scatter_ops.test_masked_scatter[2d_masked_scatter]❌ failure

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests590 ran464 passed124 skipped2 failed
TestResult
TT-Forge-FE Tests
pytest
test_scatter_ops.test_masked_scatter[1d_masked_scatter]❌ failure
test_mobilenet_v2.test_mobilenetv2_basic❌ failure

ashokkumarkannan1 added a commit that referenced this pull request Feb 8, 2025
@ashokkumarkannan1
Add Llava model bringup (#1173)
b0a7d77 
### Ticket
Fix #1177

### What's changed
Add Model tests for Llava Model

### Logs

[test_llava.log](https://github.com/user-attachments/files/18687248/test_llava.log)

### Current Issue
```
E           NotImplementedError: The following operators are not implemented: ['aten::masked_scatter']
```

And this will be solved in this PR -
#1188
@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch 2 times, most recently from d828ed5 to 1b55360 Compare February 8, 2025 09:33
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

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

1 similar comment
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

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

@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 8, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

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

1 similar comment
@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

1 similar comment
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

@vkovinicTT
Copy link
Member

Let's first finish the change on the tvm side, so we can get to this later. Also please change problem description and What's changed paragraphs so that they reflect actual change, since now it is a bit confusion as we are not actually adding mapping here, but only changing the tests :))

@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch from be7363f to a36b7ad Compare February 13, 2025 15:37
@github-actions GitHub Actions
Copy link

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

1 similar comment
@github-actions GitHub Actions
Copy link

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

@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

1 similar comment
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests650 ran509 passed141 skipped0 failed
TestResult
No test annotations available

@ashokkumarkannan1
Copy link
Contributor Author

Hi @nvukobratTT & @vkovinicTT ,
Can you please review and approve this PR? I have now merged TVM's PR associated with this. Once this is approved, I will merge this one as well.

Thanks .

@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch from a36b7ad to 0d6ee4a Compare February 25, 2025 07:12
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests665 ran522 passed143 skipped0 failed
TestResult
No test annotations available

1 similar comment
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests665 ran522 passed143 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests606 ran480 passed125 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_dla.test_dla_pytorch[dla34]❌ failure

@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests606 ran481 passed125 skipped0 failed
TestResult
No test annotations available

1 similar comment
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests606 ran481 passed125 skipped0 failed
TestResult
No test annotations available

@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch 4 times, most recently from feab4f4 to cc71366 Compare February 25, 2025 16:41
@ashokkumarkannan1 ashokkumarkannan1 force-pushed the akannan/op_support_maskedscatter branch from cc71366 to 4a997b5 Compare February 25, 2025 16:47
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests672 ran538 passed134 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

TestsPassed ☑️Skipped ⚠️Failed ❌️
TT-Forge-FE Tests614 ran479 passed134 skipped1 failed
TestResult
TT-Forge-FE Tests
pytest
test_dla.test_dla_pytorch[dla34]❌ failure

@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests672 ran538 passed134 skipped0 failed
TestResult
No test annotations available

@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests614 ran480 passed134 skipped0 failed
TestResult
No test annotations available

1 similar comment
@github-actions GitHub Actions
Copy link

TestsPassed ✅Skipped ⚠️Failed
TT-Forge-FE Tests614 ran480 passed134 skipped0 failed
TestResult
No test annotations available

@ashokkumarkannan1 ashokkumarkannan1 merged commit 5b2ebf2 into main Feb 26, 2025
10 checks passed
@ashokkumarkannan1 ashokkumarkannan1 deleted the akannan/op_support_maskedscatter branch February 26, 2025 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nvukobratTT nvukobratTT nvukobratTT approved these changes

@pilkicTT pilkicTT Awaiting requested review from pilkicTT pilkicTT is a code owner

@dgolubovicTT dgolubovicTT Awaiting requested review from dgolubovicTT dgolubovicTT is a code owner

@vkovinicTT vkovinicTT Awaiting requested review from vkovinicTT

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NotImplementedError: The following operators are not implemented: ['aten::masked_scatter']
3 participants
@ashokkumarkannan1 @nvukobratTT @vkovinicTT