lonelyeagle opened a new pull request, #18888: URL: https://github.com/apache/tvm/pull/18888
This PR fixes an edge case in OnnxOpConverter.get_converter in python/tvm/relax/frontend/onnx/onnx_frontend.py. Previously, when the requested opset was lower than the minimum supported _impl_v* version, the python list index logic could select an invalid previous element (via negative indexing) instead of failing clearly. This change keeps the existing converter-selection logic and adds a guard: compute the index of opset in the sorted version list (including opset) if opset is at index 0, raise NotImplementedError otherwise continue with existing behavior This ensures unsupported low opset versions fail fast with an explicit error instead of potentially selecting an incorrect converter. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
