Skip to content

[BYOC][VitisAI] Fix issue in Vitis AI codegen out tensor names matching & update docs and docker#7350

Merged
comaniac merged 10 commits intoapache:mainfrom
anilmartha:vitis-ai-fix-out-tensors-order2
Feb 24, 2021
Merged

[BYOC][VitisAI] Fix issue in Vitis AI codegen out tensor names matching & update docs and docker#7350
comaniac merged 10 commits intoapache:mainfrom
anilmartha:vitis-ai-fix-out-tensors-order2

Conversation

@jtuyls
Copy link
Copy Markdown
Contributor

@jtuyls jtuyls commented Jan 27, 2021

Fix occasional issue in Vitis AI codegen out tensor names matching.
Small updates in Vitis AI docs & demo_vitis_ai docker.

@comaniac @zhiics @anilmartha

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Comment thread docker/Dockerfile.demo_vitis_ai Outdated
Comment thread docker/install/ubuntu_install_vitis_ai_core.sh Outdated
Copy link
Copy Markdown
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
Copy link
Copy Markdown
Contributor

cc @tqchen as I think it will require a Docker images rebuild

Comment thread docs/deploy/vitis_ai.rst Outdated
Comment thread docs/deploy/vitis_ai.rst Outdated
Comment thread python/tvm/contrib/target/vitis_ai.py Outdated
@jtuyls
Copy link
Copy Markdown
Contributor Author

jtuyls commented Feb 14, 2021

@comaniac I made some adjustments based on your comments and updated to main to make the tests pass. Could you have another look at this?

Copy link
Copy Markdown
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Just a nit.

Comment thread python/tvm/contrib/target/vitis_ai.py Outdated
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Comment thread docs/deploy/vitis_ai.rst
@comaniac comaniac merged commit 1455536 into apache:main Feb 24, 2021
@comaniac
Copy link
Copy Markdown
Contributor

Thanks @jtuyls @leandron @zhiics

Lokiiiiii pushed a commit to Lokiiiiii/tvm that referenced this pull request Mar 2, 2021
…ng & update docs and docker (apache#7350)

* Fix bug in vitis ai codegen out tensor names matching & update docs & update docker

* Update vitis_ai.rst

* Move gpg-agent package installation to vitis ai core script

* Refactor install_vitis_ai_core script

* Update docs/deploy/vitis_ai.rst

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update docs/deploy/vitis_ai.rst

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update vitis-ai docs pynq/edge setup & adjustements for comments

* Update python/tvm/contrib/target/vitis_ai.py

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Reorg Vitis AI dockerfile to make sure gpg-agent is installed before llvm

Co-authored-by: Jorn Tuyls <jornt.tuyls@gmail.com>
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Mar 2, 2021
…ng & update docs and docker (apache#7350)

* Fix bug in vitis ai codegen out tensor names matching & update docs & update docker

* Update vitis_ai.rst

* Move gpg-agent package installation to vitis ai core script

* Refactor install_vitis_ai_core script

* Update docs/deploy/vitis_ai.rst

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update docs/deploy/vitis_ai.rst

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update vitis-ai docs pynq/edge setup & adjustements for comments

* Update python/tvm/contrib/target/vitis_ai.py

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Reorg Vitis AI dockerfile to make sure gpg-agent is installed before llvm

Co-authored-by: Jorn Tuyls <jornt.tuyls@gmail.com>
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants