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/12/05 18:22:39 UTC

[GitHub] aaronmarkham commented on issue #13541: Updated docs for randint operator

aaronmarkham commented on issue #13541: Updated docs for randint operator
URL: https://github.com/apache/incubator-mxnet/pull/13541#issuecomment-444589338
 
 
   Here's the current output at: http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-13541/3/api/python/ndarray/ndarray.html#random-sampling
   
   ![2018-12-05_10-16-35](https://user-images.githubusercontent.com/5974205/49534701-1613e780-f877-11e8-8a0d-2d7930b7a837.png)
   
   First thing I'm noticing is that you probably could have it showing minus `mxnet.ndarray` so each entry is like `random.x`. Note how the other sections are like this.
   
   I see that `mxnet.random.seed` is different and not part of ndarray, so it make sense to have the full name.
   
   Then within this `Random Sampling` category, the entries should be alphabetical. Rooting around in a long list for the function you want is difficult when it isn't alphabetical.
   

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