[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-12-05 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-562461420 @wkcn Big thank to Jackie for the merging work! This is an automated

[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-10-24 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-545776178 @wkcn 1.6 code freeze is tomorrow, so are you ok with this one not going into the 1.6 release? It is because none of us have time to

[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-10-16 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-542954858 @szha we can also copy the implementation of dmlc::registry & dlpack to lib_api.h, but it doesn't solve the diverge problem you described.

[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-10-16 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-542951984 @szha Because we want to make lib_api.h self-contained without any dependencies. The end goal is that the user will only copy lib_api.h to

[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-10-14 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-541985445 talked with @szha offline and we agree on putting DLTensor support to the next PR

[GitHub] [incubator-mxnet] rondogency commented on issue #15921: dynamic custom operator support

2019-10-02 Thread GitBox
rondogency commented on issue #15921: dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-537376762 @samskalicky @wkcn @szha @junrushao1994 @eric-haibin-lin This PR is ready for review. c_api.cc and lib_api.h files are the most