-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
python/mxnet/numpy/random.py
Outdated
------- | ||
out : ndarray | ||
A ``(d0, d1, ..., dn)``-shaped array of floating-point samples from | ||
the standard normal distribution, or a single such float if |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should remove "from the standard normal".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix it. Thank You.
python/mxnet/symbol/numpy/random.py
Outdated
------- | ||
out : ndarray | ||
A ``(d0, d1, ..., dn)``-shaped array of floating-point samples from | ||
the standard normal distribution, or a single such float if |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above.
* fix document string -> Returns description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please rebase with current master and resolve the conflicts @kshitij12345 |
@haojin2 Rebase with master or numpy branch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the contribution. Could you please rebase to resolve the conflicts?
add relevant tests address comments. * fix document string -> Returns description.
f051954
to
ba4373f
Compare
@reminisce @haojin2 @xidulu Sorry for the delayed action. Please review. |
@kshitij12345 Thanks for your contribution. It's merged through #16554. I added missing implementation in |
@reminisce Oh did not notice it. Thank You. Will close this one. |
Description
Add
https://docs.scipy.org/doc/numpy-1.16.0/reference/generated/numpy.random.rand.html
Heavily Referenced from #15903
@xidulu Thank You.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes