Skip to content

Conversation

@ANSHUMAN87
Copy link
Contributor

1: Sparse Add Op added in Topi.
2: TF Frontend support added.

@ANSHUMAN87 ANSHUMAN87 force-pushed the sparse-37 branch 5 times, most recently from d7790fa to 7ee8692 Compare February 17, 2021 07:11
@ANSHUMAN87 ANSHUMAN87 marked this pull request as ready for review February 17, 2021 16:39
@ANSHUMAN87
Copy link
Contributor Author

cc @tkonolige , @FrozenGene!

Copy link
Contributor

@tkonolige tkonolige left a 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.

@ANSHUMAN87
Copy link
Contributor Author

@tkonolige : All your comments are addressed now, PTAL. Thanks!

@ANSHUMAN87
Copy link
Contributor Author

cc @FrozenGene , @mbaret !

@ANSHUMAN87
Copy link
Contributor Author

Gentle ping @FrozenGene , @mbaret !

Copy link
Contributor

@mbaret mbaret left a 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?

@ANSHUMAN87
Copy link
Contributor Author

Broadly lgtm. I'm not especially familiar with sparse implementations. Perhaps @giuseros you could also take a quick look at the TOPI stuff?

Gentle ping @giuseros !

Copy link
Contributor

@mbaret mbaret left a 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.

@mbaret mbaret merged commit 017ff94 into apache:main Mar 11, 2021
@mbaret
Copy link
Contributor

mbaret commented Mar 11, 2021

Thanks @ANSHUMAN87, @tkonolige, this is merged.

trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
* [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
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request May 11, 2021
* [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
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.

3 participants