Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Use apply_squeeze in opset 13 rather than add_node #669

Merged
merged 9 commits into from
Dec 18, 2020
Merged

Use apply_squeeze in opset 13 rather than add_node #669

merged 9 commits into from
Dec 18, 2020

Conversation

jiafatom
Copy link
Collaborator

No description provided.

@jiafatom jiafatom changed the title Use apply_squeeze in opset 13 rather than add_node [WIP] Use apply_squeeze in opset 13 rather than add_node Dec 17, 2020
@lgtm-com
Copy link

lgtm-com bot commented Dec 17, 2020

This pull request introduces 4 alerts when merging a25e713 into 16a42f5 - view on LGTM.com

new alerts:

  • 2 for Variable defined multiple times
  • 1 for Unused local variable
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Dec 18, 2020

This pull request introduces 1 alert when merging d5c54ea into 16a42f5 - view on LGTM.com

new alerts:

  • 1 for Unused local variable

@jiafatom jiafatom changed the title [WIP] Use apply_squeeze in opset 13 rather than add_node Use apply_squeeze in opset 13 rather than add_node Dec 18, 2020
@jiafatom jiafatom merged commit 680afce into onnx:master Dec 18, 2020
@jiafatom jiafatom deleted the apply_squeeze_13 branch December 18, 2020 11:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@wenbingl wenbingl wenbingl approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jiafatom @wenbingl