Skip to content

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

Merged
jcf94 merged 6 commits intoapache:mainfrom
mbrookhart:onnx_scatter_nd
Apr 28, 2021
Merged

[TOPI][RELAY][ONNX] Scatter ND#7927
jcf94 merged 6 commits intoapache:mainfrom
mbrookhart:onnx_scatter_nd

Conversation

@mbrookhart
Copy link
Copy Markdown
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
Comment thread python/tvm/topi/cuda/scatter.py Outdated
Comment thread python/tvm/topi/x86/scatter.py Outdated
Comment thread python/tvm/topi/cuda/scatter.py Outdated
Comment thread python/tvm/topi/scatter.py Outdated
jwfromm
jwfromm approved these changes Apr 26, 2021
View reviewed changes
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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