Skip to content

[Relay][Runtime] Add set_input/output_zero_copy in python#13623

Merged
masahi merged 16 commits intoapache:mainfrom
shingjan:add_apis_gm
Dec 17, 2022
Merged

[Relay][Runtime] Add set_input/output_zero_copy in python#13623
masahi merged 16 commits intoapache:mainfrom
shingjan:add_apis_gm

Conversation

@shingjan
Copy link
Copy Markdown

As titled, this apis is in cpp but not python. Added this apis for GraphModule and some tests.

cc: @vinx13

@tvm-bot
Copy link
Copy Markdown
Collaborator

tvm-bot commented Dec 15, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

Comment thread python/tvm/contrib/graph_executor.py Outdated
Comment thread src/runtime/graph_executor/graph_executor.cc Outdated
Comment thread python/tvm/contrib/graph_executor.py Outdated
Comment thread python/tvm/contrib/graph_executor.py Outdated
Comment thread python/tvm/contrib/graph_executor.py
@shingjan shingjan requested a review from vinx13 December 15, 2022 18:54
Comment thread python/tvm/contrib/graph_executor.py Outdated
@shingjan shingjan requested a review from vinx13 December 15, 2022 19:29
@shingjan
Copy link
Copy Markdown
Author

@tvm-bot re-run

Comment thread tests/python/unittest/test_runtime_module_based_interface.py
@shingjan shingjan requested a review from echuraev December 16, 2022 22:09
@masahi masahi merged commit 564934d into apache:main Dec 17, 2022
fzi-peccia pushed a commit to fzi-peccia/tvm that referenced this pull request Mar 27, 2023
…3623)

* add set_output and test for set_output_zero_copy in python

* clean up

* clean up test

* test finished

* remove set output

* remove setoutput from header

* use zero copy for params

* fix typo

* address comments

* address comments

* add second test for set_input params

* add requires_torch

* add requires torch

* remove pytest

* add error handling for c graph executor

* better handling
mikeseven pushed a commit to mikeseven/tvm that referenced this pull request Sep 27, 2023
…3623)

* add set_output and test for set_output_zero_copy in python

* clean up

* clean up test

* test finished

* remove set output

* remove setoutput from header

* use zero copy for params

* fix typo

* address comments

* address comments

* add second test for set_input params

* add requires_torch

* add requires torch

* remove pytest

* add error handling for c graph executor

* better handling
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.

5 participants