-
Notifications
You must be signed in to change notification settings - Fork 683
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
build(tvm_utility): remove download logic from CMake and update documentation #4923
Merged
esteve
merged 59 commits into
autowarefoundation:main
from
lexavtanke:remove_download_tvm_utility
Oct 26, 2023
Merged
Changes from 7 commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
c90b228
add include tier4_autoware_utils and dependency
lexavtanke e77245c
remove downloading logic from Cmake, update documentation
lexavtanke 59c590f
build(tvm_utility): remove downloading logic from Cmake, update docum…
lexavtanke deab5e7
Merge remote-tracking branch 'lexavtanke/remove_download_tvm_utility'…
lexavtanke d4e0064
style(pre-commit): autofix
pre-commit-ci[bot] a098b48
build(tvm_utility): fix lint_cmake error
lexavtanke dac7a20
build(tvm_utility): format warning message
lexavtanke 789ba72
build(tvm_utility): add logic to work with autoware_data folder, add …
lexavtanke 0fa9061
style(pre-commit): autofix
pre-commit-ci[bot] 4060e78
Merge branch 'autowarefoundation:main' into remove_download_tvm_utility
lexavtanke 726e3a2
style(pre-commit): autofix
pre-commit-ci[bot] f9d23be
Merge branch 'main' into remove_download_tvm_utility
lexavtanke a602a34
build(tvm_utility): refactor, update InferenceEngineTVM constructor
lexavtanke e2ad796
style(pre-commit): autofix
pre-commit-ci[bot] 1c46e02
Merge branch 'autowarefoundation:main' into remove_download_tvm_utility
lexavtanke b218bbe
build(tvm_utility): add lightweight model and test with it
lexavtanke 6eea3e9
build(tvm_utility): make building yolo_v2_tiny disable by default
lexavtanke dd09064
build(tvm_utility): remove test artifact for yolo_v2_tiny
lexavtanke 3cf6108
build(tvm_utility): update docs
lexavtanke e82b412
build(tvm_utility): update docs
lexavtanke 5ed5724
style(pre-commit): autofix
pre-commit-ci[bot] 7ba6056
build(tvm_utility): update namespace in abs_model test
lexavtanke c45ab80
build(tvm_utility): rewrite yolo_v2_tiny as example
lexavtanke c91c0e2
build(tvm_utility): clean comments in yolo_v2_tiny example main.cpp
lexavtanke f145dbc
build(tvm_utility): add launch file for yolo_v2_tiny example
lexavtanke 6d86664
build(tvm_utility): update yolo_v2_tiny example readme
lexavtanke 24aee4b
style(pre-commit): autofix
pre-commit-ci[bot] 3484369
build(tvm_utility): add model for arm based systems, need to be teste…
lexavtanke 165255a
style(pre-commit): autofix
pre-commit-ci[bot] 0c08dc3
Merge branch 'autowarefoundation:main' into remove_download_tvm_utility
lexavtanke 603c163
style(pre-commit): autofix
pre-commit-ci[bot] dd8e24a
build(tvm_utility): update config header for arm
lexavtanke 971508a
style(pre-commit): autofix
pre-commit-ci[bot] d6992b1
build(tvm_utility): remove debug output
lexavtanke 8f2f263
Merge branch 'main' into remove_download_tvm_utility
lexavtanke 107c939
build(tvm_utility): add find_package conditional section
lexavtanke e360caf
build(tvm_utility): fix lint_cmake errors
lexavtanke 99caa44
build(tvm_utility): remove coping model files during build
lexavtanke eb3b8bb
build(tvm_utility): update readme with new data folder structure
lexavtanke 63e932a
build(tvm_utility): fix spell check warnings
lexavtanke 1b22105
style(pre-commit): autofix
pre-commit-ci[bot] 4b8471d
build(tvm_utility): add no model files guard to get_neural_network
lexavtanke 3514fd9
style(pre-commit): autofix
pre-commit-ci[bot] c50d1df
build(tvm_utility): set back default paths in config headers
lexavtanke 770c400
build(tvm_utility): add param file, update launch file
lexavtanke 0dd4431
build(tvm_utility): add schema file, update node name
lexavtanke 9fced41
style(pre-commit): autofix
pre-commit-ci[bot] a8b65b2
build(tvm_utility): fix json-schema-check
lexavtanke 742903f
Merge remote-tracking branch 'lexavtanke/remove_download_tvm_utility'…
lexavtanke 7999377
build(tvm_utility): fix json-schema-check
lexavtanke ab86d1f
style(pre-commit): autofix
pre-commit-ci[bot] 3300af7
build(tvm_utility): add parameter table to example readme
lexavtanke 81f9aed
build(tvm_utility): fix typo-error in description of schema.json
lexavtanke 70cabe2
style(pre-commit): autofix
pre-commit-ci[bot] 0d81afb
Merge branch 'main' into remove_download_tvm_utility
lexavtanke d080050
buiild(tvm_utility): fix spell-check warning and typo
lexavtanke 2f190c6
feat(spell-check): add dltype and tvmgen to local dict
lexavtanke 4f7d1fe
Merge branch 'feat-spell-check-add-tvmg-and-dltype' into remove_downl…
lexavtanke 3c97359
style(pre-commit): autofix
pre-commit-ci[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the artifacts are required for compilation, then they should be part of the source tree (via
git-lfs
for example if the files are too big), otherwise it won't be possible to build Debian packages.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually It is a bit confusing for now. Because this packages don't seem to build anyway now because DOWNLOAD_ARTIFACTS flag is disabled by default and this PR kinda show it because last changes was made several months ago and bug was hidden. So here I just follow the same logic.
And as I understand before we are going to use ansible to provide artifacts. And for packages with tenosorrt support It is more or less straight forward process as all model conversion from onnx to TRT happens later on the first run not on the build.
For TVM it is a bit more complicated as it provide models in already compiled form and to use it in the package you will need to provide some model files to build it.
But if I understand you correctly here. You propose that there should be some default artifacts for all TVM packages which should be part of the source tree and the only way to do it is
git-lfs
because for nowdeploy_param.params
file weights 10-20 Mb usually. And If user will want to provide his own models wouldn't it be messy a bit ?I think there is a kinda similar way to complile the model to TVM before run by the user same as with TRT but I'm not so familiar with TVM.
And what is the idea behind the Debian packages? Should user be able to use them of the shelf with the build in model? I guess in many cases user with have to use model trained on his own data.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I know, we only need the headers for building the packages, not the full models.
The Debian packages are a way for users to install Autoware in production environments without having to build it themselves, much like how ROS is distributed. Eventually, Autoware can become part of the ROS distribution, but that will come at a later stage.