You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/03/15 16:30:53 UTC
[GitHub] [tvm] jtuyls opened a new pull request #7665: [Runtime] Fix GraphRuntime.load_params to allow passing parameters that are not an input
jtuyls opened a new pull request #7665:
URL: https://github.com/apache/tvm/pull/7665
Fix for `TVMArrayCopyFromTo` error that happens when parameters that are not expected inputs are loaded using GraphRuntime::LoadParams. See the added test cases for a BYOC flow example that exposes this issue. @tkonolige @junrushao1994 @areusch
```
tvm._ffi.base.TVMError: Traceback (most recent call last):
E [bt] (8) /workspace/build/libtvm.so(tvm::runtime::PackedFunc::CallPacked(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) const+0x30) [0x7f018b8c7c46]
E [bt] (7) /workspace/build/libtvm.so(std::function<void (tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)>::operator()(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) const+0x5a) [0x7f018b75c66c]
E [bt] (6) /workspace/build/libtvm.so(+0x2d38962) [0x7f018ca30962]
E [bt] (5) /workspace/build/libtvm.so(+0x2d35f66) [0x7f018ca2df66]
E [bt] (4) /workspace/build/libtvm.so(tvm::runtime::GraphRuntime::LoadParams(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)+0x46) [0x7f018ca2a742]
E [bt] (3) /workspace/build/libtvm.so(tvm::runtime::GraphRuntime::LoadParams(dmlc::Stream*)+0x157) [0x7f018ca2a8dd]
E [bt] (2) /workspace/build/libtvm.so(tvm::runtime::NDArray::CopyFrom(tvm::runtime::NDArray const&)+0x158) [0x7f018ca32b7c]
E [bt] (1) /workspace/build/libtvm.so(tvm::runtime::NDArray::CopyFromTo(DLTensor const*, DLTensor*, void*)+0x120) [0x7f018c9b1876]
E [bt] (0) /workspace/build/libtvm.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x4e) [0x7f018b6cc6fa]
E File "/workspace/src/runtime/ndarray.cc", line 230
E TVMError:
E ---------------------------------------------------------------
E An internal invariant was violated during the execution of TVM.
E Please read TVM's error reporting guidelines.
E More details can be found here: https://discuss.tvm.ai/t/error-reporting/7793.
E ---------------------------------------------------------------
E
E Check failed: from_size == to_size (4 vs. 40) : TVMArrayCopyFromTo: The size must exactly match
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] comaniac merged pull request #7665: [Runtime] Fix GraphRuntime.load_params to allow passing parameters that are not an input
Posted by GitBox <gi...@apache.org>.
comaniac merged pull request #7665:
URL: https://github.com/apache/tvm/pull/7665
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] comaniac commented on pull request #7665: [Runtime] Fix GraphRuntime.load_params to allow passing parameters that are not an input
Posted by GitBox <gi...@apache.org>.
comaniac commented on pull request #7665:
URL: https://github.com/apache/tvm/pull/7665#issuecomment-804677134
Thanks @jtuyls @tkonolige
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org