jwfromm opened a new pull request #4825: [Frontend][ONNX] LSTM Support URL: https://github.com/apache/incubator-tvm/pull/4825 This PR adds LSTM support to the relay Onnx frontend. Besides adding the LSTM parser itself, we encountered an issue where for some Onnx operations (like LSTMs) arguments are optional. The current method for passing arguments to converters is just to pack them into a list however as some arguments are optional the position of each input is inconsistent. Instead, we should be using a dictionary mapping input names to their value. However, changing all inputs to a dictionary would require changing all the current operators and present problems with direct Onnx to relay conversions. Our workaround here is to add the `onnx_input` class that can be accessed as a list as we previously did or with input name dictionary style lookup.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
