You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/03/28 14:27:44 UTC
[GitHub] zheng-da commented on issue #10282: plot_network issue with LRN
zheng-da commented on issue #10282: plot_network issue with LRN
URL: https://github.com/apache/incubator-mxnet/issues/10282#issuecomment-376906705
I'll fix the LRN op.
@reminisce do we really need to provide output names? It seems the old interface doesn't provide output names, except for the operators that have multiple outputs.
BTW, it seems your fix has a bug.
https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/convolution.cc#L471
https://github.com/apache/incubator-mxnet/blob/master/src/operator/nn/fully_connected.cc#L270
Could you please fix it? It should use FListOutputNames.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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