[VTA] Improved RPC for VTA#2043
Merged
tmoreau89 merged 10 commits intoapache:masterfrom Nov 12, 2018
Merged
Conversation
`build/vta_config.json` => `vta/config/vta_config.json`
tmoreau89
reviewed
Oct 31, 2018
Member
Author
|
@tmoreau89 Thank you for your instructions. I have made the following change to reflect the design decision:
On the other hand, since the source code under |
Contributor
|
@liangfu renaming the metal tests is a good idea since the aim is to have mostly a device independent workflow |
Member
Author
|
@tmoreau89 renamed and pushed, please review. |
tmoreau89
requested changes
Nov 12, 2018
Contributor
tmoreau89
left a comment
tmoreau89
left a comment
There was a problem hiding this comment.
please undo the change in rpc_server.py, but otherwise, it's ready to merge!
FrozenGene
pushed a commit
to FrozenGene/tvm
that referenced
this pull request
Dec 27, 2018
* assign default port to 9091 as the documented * bug fix in printing RuntimeError and add an additional search path * pretty print rebuild runtime args * PRC => RPC * replace vta_config.json file path `build/vta_config.json` => `vta/config/vta_config.json` * undo the change in adding lib_search path * search vta_config.py file in vta/config * avoid exposing driver function calls, and use predefined `VTAMemGetPhyAddr` instead. * rename `tests/hardware/pynq` => `metal_test` * set config path back to `build` dir
Closed
wweic
pushed a commit
to neo-ai/tvm
that referenced
this pull request
Feb 20, 2019
* assign default port to 9091 as the documented * bug fix in printing RuntimeError and add an additional search path * pretty print rebuild runtime args * PRC => RPC * replace vta_config.json file path `build/vta_config.json` => `vta/config/vta_config.json` * undo the change in adding lib_search path * search vta_config.py file in vta/config * avoid exposing driver function calls, and use predefined `VTAMemGetPhyAddr` instead. * rename `tests/hardware/pynq` => `metal_test` * set config path back to `build` dir
wweic
pushed a commit
to neo-ai/tvm
that referenced
this pull request
Feb 20, 2019
* assign default port to 9091 as the documented * bug fix in printing RuntimeError and add an additional search path * pretty print rebuild runtime args * PRC => RPC * replace vta_config.json file path `build/vta_config.json` => `vta/config/vta_config.json` * undo the change in adding lib_search path * search vta_config.py file in vta/config * avoid exposing driver function calls, and use predefined `VTAMemGetPhyAddr` instead. * rename `tests/hardware/pynq` => `metal_test` * set config path back to `build` dir
tqchen
pushed a commit
to tqchen/tvm
that referenced
this pull request
Mar 29, 2020
* assign default port to 9091 as the documented * bug fix in printing RuntimeError and add an additional search path * pretty print rebuild runtime args * PRC => RPC * replace vta_config.json file path `build/vta_config.json` => `vta/config/vta_config.json` * undo the change in adding lib_search path * search vta_config.py file in vta/config * avoid exposing driver function calls, and use predefined `VTAMemGetPhyAddr` instead. * rename `tests/hardware/pynq` => `metal_test` * set config path back to `build` dir
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Changes
libvta.so@tmoreau89 would you please review this PR?