Skip to content
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

[ONNX] Update slice to infer attributes when not graph inputs #6276

Merged
merged 2 commits into from
Aug 17, 2020
Merged

[ONNX] Update slice to infer attributes when not graph inputs #6276

merged 2 commits into from
Aug 17, 2020

Conversation

csullivan
Copy link
Contributor

@csullivan csullivan commented Aug 14, 2020

The slice converter was only checking for its input attributes in the list of graph parameters which fails when the attributes are calculated as is the case in DeeplabV3 (MNv2). Update the importer to infer the slice attribute values when this is the case.

@csullivan csullivan changed the title [ONNX] Update slice to infer bound attributes when not graph inputs [ONNX] Update slice to infer attributes when not graph inputs Aug 14, 2020
@csullivan csullivan marked this pull request as ready for review August 14, 2020 18:38
@csullivan
Copy link
Contributor Author

cc @mbrookhart @jwfromm

Copy link
Contributor

@mbrookhart mbrookhart left a comment

Choose a reason for hiding this comment

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

LGTM

cc @masahi

@masahi masahi merged commit 4644991 into apache:master Aug 17, 2020
@masahi
Copy link
Member

masahi commented Aug 17, 2020

Thanks @csullivan @mbrookhart

trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
…#6276)

* Update ONNX Slice converter to infer slice attributes when necessary.

* Linting
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
…#6276)

* Update ONNX Slice converter to infer slice attributes when necessary.

* Linting
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
…#6276)

* Update ONNX Slice converter to infer slice attributes when necessary.

* Linting
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Sep 2, 2020
…#6276)

* Update ONNX Slice converter to infer slice attributes when necessary.

* Linting
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Sep 3, 2020
…#6276)

* Update ONNX Slice converter to infer slice attributes when necessary.

* Linting
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