Skip to content

[TOPI][RELAY][ONNX] Scatter ND #7927

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
Merged
merged 6 commits into from
Apr 28, 2021
Merged

[TOPI][RELAY][ONNX] Scatter ND #7927

merged 6 commits into from
Apr 28, 2021

Conversation

@mbrookhart
Copy link
Contributor

@mbrookhart mbrookhart commented Apr 26, 2021

This PR refactors the relay scatter_nd op in two ways:

  1. scatter_nd now takes in a data tensor, and updates are scattered based on indices. The previous API assumed zero-initialization for the data.
  2. scatter_nd now has a "mode" argument to determine how to update the input data, either "add" or "update". This is to support the use cases as a gradient of gather_nd and to support the ONNX and pytorch APIs. TF also supports "max", "min", and "sub", which are not yet supported here.

This also adds ScatterND to the ONNX importer.

cc @tkonolige @altanh @jwfromm @masahi

tkonolige
tkonolige reviewed Apr 26, 2021
View reviewed changes
jwfromm
jwfromm approved these changes Apr 26, 2021
View reviewed changes
Copy link
Contributor

@jwfromm jwfromm left a comment

Choose a reason for hiding this comment

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

Very nice, LGTM.

altanh
altanh approved these changes Apr 27, 2021
View reviewed changes
Copy link
Contributor

@altanh altanh left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@jcf94 jcf94 merged commit 8fce895 into apache:main Apr 28, 2021
@jcf94
Copy link
Contributor

jcf94 commented Apr 28, 2021

Thanks! @mbrookhart @tkonolige @jwfromm @altanh

@mbrookhart mbrookhart deleted the onnx_scatter_nd branch April 28, 2021 16:13
echuraev pushed a commit to echuraev/tvm that referenced this pull request Apr 29, 2021
@echuraev
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
d55a621 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
umangyadav pushed a commit to umangyadav/tvm that referenced this pull request May 5, 2021
@umangyadav
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
df60c1f 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
859f48d 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
89bff85 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
70a8336 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request May 11, 2021
@trevor-m
[TOPI][RELAY][ONNX] Scatter ND (apache#7927)
cd7852f 
* passing topi tests

* passing relay tests, needs better shape checking still

* support ONNX operator

* add shape checking back in

* fix lint

* update docstring
@junrushao junrushao mentioned this pull request Nov 1, 2021
Closed
@JR4er JR4er mentioned this pull request May 6, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tkonolige tkonolige tkonolige left review comments

@jwfromm jwfromm jwfromm approved these changes

+1 more reviewer

@altanh altanh altanh approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mbrookhart @jcf94 @tkonolige @altanh @jwfromm