[GitHub] [incubator-mxnet] stephenrawls commented on issue #14208: Add support for fast variable-length LSTM

[GitBox <gi...@apache.org>]

stephenrawls commented on issue #14208: Add support for fast variable-length LSTM
URL: https://github.com/apache/incubator-mxnet/pull/14208#issuecomment-489296816
 
 
   @szha  -- ok I fixed the issues I was having with the linter / unit tests.
   
   The unix-gpu test is failing but it looks entirely unrelated to this PR:
   ```
   Run LeNet-5 inference comparison between MXNet and TensorRT. ... [03:47:12] /work/mxnet/src/operator/subgraph/build_subgraph.cc:683: start to execute partition graph.
   FAIL
   ======================================================================
   FAIL: Run LeNet-5 inference comparison between MXNet and TensorRT.
   ----------------------------------------------------------------------
   Traceback (most recent call last):
     File "/usr/local/lib/python3.6/dist-packages/nose/case.py", line 198, in runTest
       self.test(*self.arg)
     File "/work/mxnet/tests/python/tensorrt/test_tensorrt_lenet5.py", line 100, in test_tensorrt_inference
       MXNet = %f, TensorRT = %f""" % (mx_pct, trt_pct)
   AssertionError: Diff. between MXNet & TensorRT accuracy too high:
              MXNet = 99.020000, TensorRT = 99.030000
   ```
   
   Also the unix-cpu test is failing, and it also seems unrelated:
   
   ```
   FAIL: test_operator.test_custom_op_exc
   ...
   mxnet.base.MXNetError: [03:36:22] src/operator/tensor/./dot-inl.h:1237: Check failed: L[!Ta].Size() == R[Tb].Size() (2 vs. 1) : dot shape error: [4,2] X [1,4]
   …
   ```

----------------------------------------------------------------
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