You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/06/25 00:00:10 UTC

[GitHub] [incubator-mxnet] KellenSunderland edited a comment on issue #15344: [MXNET-1086] added sub and mul to ONNX->TensorRT conversion

KellenSunderland edited a comment on issue #15344: [MXNET-1086] added sub and mul to ONNX->TensorRT conversion
URL: https://github.com/apache/incubator-mxnet/pull/15344#issuecomment-505224822
 
 
   So I was going to point out that we'd need to update the subgraph selector to add support for the mul / sub but it seems like that's already there.  This may actually fix model conversions for models that have those ops.  LGTM, any thoughts from your side @Caenorst?
   
   https://github.com/apache/incubator-mxnet/blob/master/src/operator/subgraph/tensorrt/tensorrt-inl.h#L93

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services