Skip to content

MIGraphX doesn't report supported Onnx operations in the API #4024

@TedThemistokleous

Description

@TedThemistokleous

The API allows us to parse in onnx and tensor flow operators but we don't have a way to query the currently installed MIGraphX version to determine what operators or supported

Our MIGraphX driver has this functionality via the 'migarphx-driver onnx --list` but this isn't broken out in our API

In Onnxruntime we currently manage a list of supported OPs manually instead of querying the API on model node, if parsed in operators are supported.

If we have a list of supported OPs we can ensure we have better coverage and identification of contrib ops, while not needing to constantly updated MIGraphx Execution provider when newer support is added, along with upstreaming changes to mainline for one support is then dictated by MIGraphX version

Certain Optimizations in Onnxruntime produce "contrib" operators. This is something we haven't looked into yet and support on a case by case basis currently:
https://github.com/microsoft/onnxruntime/blob/main/docs/ContribOperators.md

Any new operators would also be easily identified and likely backed out if they produce a state where we don't support the op since we'd query the list of MIGraphX support before cutting a graph and falling back to another EP.

  • Break out get_onnx_operators() to MIGraphX API
  • Use this get_supported_onnx_operators() in MIGraphX EP

Metadata

Metadata

Labels

Onnx OperatorsAdding or modifying an Onnx Operator in the MIGraphX codebaseenhancementNew feature or requestonnxissues related to onnx supportonnxruntimePR changes interaction between MIGraphX and Onnxruntime

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions