[GitHub] [incubator-mxnet] ciyongch commented on pull request #17959: [MKLDNN] Add LSTMP to v1.6.x

2020-05-07 Thread GitBox
ciyongch commented on pull request #17959: URL: https://github.com/apache/incubator-mxnet/pull/17959#issuecomment-625589023 Thanks for your understand and help making it clear @TaoLv @ChaiBapchya . This is an automated

[GitHub] [incubator-mxnet] ciyongch commented on pull request #17959: [MKLDNN] Add LSTMP to v1.6.x

2020-05-07 Thread GitBox
ciyongch commented on pull request #17959: URL: https://github.com/apache/incubator-mxnet/pull/17959#issuecomment-625112554 >Backport to 1.6 should be included by default in 1.7 right? Hi @ChaiBapchya , not sure if it's a common practice to do so. As there're several new features

[GitHub] [incubator-mxnet] ciyongch commented on pull request #17959: [MKLDNN] Add LSTMP to v1.6.x

2020-05-07 Thread GitBox
ciyongch commented on pull request #17959: URL: https://github.com/apache/incubator-mxnet/pull/17959#issuecomment-625061521 I suggest to leave this new feature to the next release version (which requires upgrade oneDNN (ex: DNNL)), as the current version 1.7.0 is code freeze and suppose