[GitHub] [incubator-mxnet] aaronmarkham commented on issue #17959: [MKLDNN] Add LSTMP to v1.6.x

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

aaronmarkham commented on issue #17959: [MKLDNN] Add LSTMP to v1.6.x
URL: https://github.com/apache/incubator-mxnet/pull/17959#issuecomment-610517907
 
 
   > The issue related to R package test is fixed with importing the function first, using `library(methods)`
   > I've made the change to v1.6.x branch [testing it locally now].
   > 
   > Referring the doc for Reproducing the tests : https://cwiki.apache.org/confluence/display/MXNET/Reproducing+test+results
   > 
   > 1. Requirements
   > 
   > ```
   > pip3 install -r ci/requirements.txt --user
   > ```
   > 
   > 1. Build
   > 
   > ```
   > ci/build.py --docker-registry mxnetci --platform ubuntu_cpu --docker-build-retries 3 --shm-size 500m /work/runtime_functions.sh build_ubuntu_cpu_openblas
   > ```
   > 
   > 1. Test
   > 
   > ```
   > ci/build.py --docker-registry mxnetci --platform ubuntu_cpu --docker-build-retries 3 --shm-size 500m /work/runtime_functions.sh unittest_ubuntu_cpu_R
   > ```
   
   R docs are broken on 1.6.x so maybe this fix can be applied to that pipeline too?
   https://github.com/apache/incubator-mxnet/issues/17920

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