microsoft / onnxruntime

ONNX Runtime: cross-platform, high performance ML inferencing and training accelerator
https://onnxruntime.ai
MIT License
14.55k stars 2.91k forks source link

Remove reference to internals in torch.onnx #11901

Open justinchuby opened 2 years ago

justinchuby commented 2 years ago

Describe the bug

There are reference to internal names in torch.onnx. There are not meant for external consumption and should be removed. Please comment if there are things we should expose from torch.onnx.

Urgency

When we want to support torch 1.12.

jingyanwangms commented 2 years ago

Do we need _export_onnx_opset_version, _get_tensor_dim_size, _get_tensor_sizes, parse_args from torch.onnx.symbolic_helper? Tagging people who recently changed lines with symbolic_helper methods @iK1D @garymm