-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[TOPI] Sparse Add Op added #7435
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
Conversation
d7790fa to
7ee8692
Compare
|
cc @tkonolige , @FrozenGene! |
tkonolige
left a comment
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.
Looks pretty good @ANSHUMAN87. I appreciate the docs. I've left a couple of improvements.
4a535aa to
4b43ed7
Compare
|
@tkonolige : All your comments are addressed now, PTAL. Thanks! |
|
cc @FrozenGene , @mbaret ! |
|
Gentle ping @FrozenGene , @mbaret ! |
mbaret
left a comment
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.
Broadly lgtm. I'm not especially familiar with sparse implementations. Perhaps @giuseros you could also take a quick look at the TOPI stuff?
mbaret
left a comment
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.
Alright let's get this in.
|
Thanks @ANSHUMAN87, @tkonolige, this is merged. |
* [TOPI] Sparse Add Op added * lint resolved * TF frontend support added * Test case added * [1] Review comment handled * [2] Review comment handled * [3] Review comment handled * [4] Review comment handled * [5] Review comment handled
* [TOPI] Sparse Add Op added * lint resolved * TF frontend support added * Test case added * [1] Review comment handled * [2] Review comment handled * [3] Review comment handled * [4] Review comment handled * [5] Review comment handled
1: Sparse Add Op added in Topi.
2: TF Frontend support added.