samskalicky commented on issue #15921: [WIP] dynamic custom operator support URL: https://github.com/apache/incubator-mxnet/pull/15921#issuecomment-526418617 I think that hurts the ease of use to force namespacing. Plus if we internally set lib1.opA then users wont know how to call mx.nd.lib1.opA or mx.nd.lib1_opA (whatever the naming scheme is). If we give them a way to set the namespace name, its still not as easy to use as mx.nd.opA. Plus we want users to be able to override internal MXNet ops. For example mx.nd.Convolution does not support FP16 on CPU. So if a user had a need for that they could write their own. So we need ops to be overwritten, and be in the mx.nd namespace. Im thinking the best solution might be to track the number of times an operator is registered and just increment the pvalue for now.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
