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 2020/07/09 22:04:29 UTC

[GitHub] [incubator-mxnet] leezu edited a comment on issue #18680: [Doc] Gluon Block/HybridBlock/SymbolBlock appears at the wrong place in the API doc

leezu edited a comment on issue #18680:
URL: https://github.com/apache/incubator-mxnet/issues/18680#issuecomment-656373159


   This should be due to missing use of `__all__` specifier in the `gluon.nn` namespace.
   
   https://github.com/apache/incubator-mxnet/blob/beafba76395e75c093f99d20ac62e38f48e91012/python/mxnet/gluon/nn/__init__.py#L22
   
   imports the Block etc into the gluon.nn namespace and now it might be undefined behavior if Sphinx will show the Block doc as part of gluon.nn or gluon namespace.
   
   Suggest to fix the `__all__` usage and improve the website broken link checker to avoid such bugs in the future
   
   Please see https://stackoverflow.com/questions/44834/can-someone-explain-all-in-python


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