Skip to content

Adapt to new pytorch onnx exporter API for dictionary #3160

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Dec 12, 2020
Merged

Adapt to new pytorch onnx exporter API for dictionary #3160

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
249c13c
Adapt to new torch export API for dictionary
jiafatom Dec 11, 2020
91484cd
Adapt to new torch export API for dictionary
jiafatom Dec 11, 2020
f6d0f08
Merge branch 'onnx_dict' of https://github.com/jiafatom/vision into o…
jiafatom Dec 11, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ jobs:
# need to install torchvision dependencies due to transitive imports
pip install --user --progress-bar off --editable .
pip install --user onnx
pip install --user -i https://test.pypi.org/simple/ ort-nightly==1.5.2.dev202012031
pip install --user onnxruntime
python test/test_onnx.py

binary_linux_wheel:
Expand Down
2 changes: 1 addition & 1 deletion .circleci/config.yml.in
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ jobs:
# need to install torchvision dependencies due to transitive imports
pip install --user --progress-bar off --editable .
pip install --user onnx
pip install --user -i https://test.pypi.org/simple/ ort-nightly==1.5.2.dev202012031
pip install --user onnxruntime
python test/test_onnx.py

binary_linux_wheel:
Expand Down
6 changes: 5 additions & 1 deletion test/test_onnx.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,12 @@ def run_model(self, model, inputs_list, tolerate_small_mismatch=False, do_consta
model.eval()

onnx_io = io.BytesIO()
if isinstance(inputs_list[0][-1], dict):
torch_onnx_input = inputs_list[0] + ({},)
else:
torch_onnx_input = inputs_list[0]
# export to onnx with the first input
torch.onnx.export(model, inputs_list[0], onnx_io,
torch.onnx.export(model, torch_onnx_input, onnx_io,
do_constant_folding=do_constant_folding, opset_version=_onnx_opset_version,
dynamic_axes=dynamic_axes, input_names=input_names, output_names=output_names)
# validate the exported model with onnx runtime
Expand Down