[GitHub] imgemp commented on issue #7410: mxnet random seed should not be fixed by default

[GitBox <gi...@apache.org>]

imgemp commented on issue #7410: mxnet random seed should not be fixed by default
URL: https://github.com/apache/incubator-mxnet/issues/7410#issuecomment-400418607
 
 
   I strongly disagree. Fixing the random seed to zero is not the default behavior in any package I've come across. It also goes against the purpose of a "random" library. By default, any outputs that are expected to exhibit randomness, should not be deterministic. When debugging or ensuring reproducibility, it is useful to be able to set the random seed, but otherwise it should be set as you say: based on time or system state. At the least, this should raise a warning. It's very difficult to discover this behavior on your own when other commonly used libraries like numpy add an additional source of true randomness on top of mxnet's deterministic default.

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