Skip to content

Conversation

@AndrewZhaoLuo
Copy link
Contributor

Response to #9519

cc @areusch

@AndrewZhaoLuo
Copy link
Contributor Author

ping @areusch

@AndrewZhaoLuo
Copy link
Contributor Author

poke @areusch

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

LGTM

@leandron leandron merged commit 8055184 into apache:main Dec 7, 2021
@leandron
Copy link
Contributor

leandron commented Dec 7, 2021

I checked and onnxoptimizer publishes packages for python 3.6 to 3.9, so it looks safe to add this dependency.

Thanks @AndrewZhaoLuo!

ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
* adds onnxoptimizer to gen_requirements.py
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 11, 2022
* adds onnxoptimizer to gen_requirements.py
yangulei pushed a commit to yangulei/tvm that referenced this pull request Jan 12, 2022
* adds onnxoptimizer to gen_requirements.py
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* adds onnxoptimizer to gen_requirements.py
qsqqsqqsq-intellif pushed a commit to qsqqsqqsq-intellif/tvm that referenced this pull request Apr 29, 2022
* adds onnxoptimizer to gen_requirements.py
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.

2 participants