-
Notifications
You must be signed in to change notification settings - Fork 626
[TorchToLinalg] : Lower count_nonzero to Linalg-on-Tensors #4145
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
[TorchToLinalg] : Lower count_nonzero to Linalg-on-Tensors #4145
Conversation
0d5418c
to
441c238
Compare
Hi @vivekkhandelwal1 , could you please approve the workflows when you get the chance? Thanks in advance! |
441c238
to
6525c97
Compare
Hi @vivekkhandelwal1 , I’ve rebased the branch and the CI workflows need maintainer approval again. Could you please click “Approve and run” when you have a moment? Thanks! |
projects/pt1/python/torch_mlir_e2e_test/test_suite/reduction.py
Outdated
Show resolved
Hide resolved
projects/pt1/python/torch_mlir_e2e_test/test_suite/reduction.py
Outdated
Show resolved
Hide resolved
projects/pt1/python/torch_mlir_e2e_test/test_suite/reduction.py
Outdated
Show resolved
Hide resolved
6525c97
to
d474f6b
Compare
Thanks for the review, @vivekkhandelwal1! I think I have addressed all requested changes—please let me know if there’s anything else to fix. |
d474f6b
to
79c50e1
Compare
79c50e1
to
1573ce3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @vivekkhandelwal1 ! Could you, please, help me merge it? I don’t have the permissions. Much appreciated! |
Uh oh!
There was an error while loading. Please reload this page.