[GitHub] [incubator-mxnet] haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)

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

haojin2 commented on issue #17255: [DO NOT REVIEW]Set np default dtype( float32 <-> float64)
URL: https://github.com/apache/incubator-mxnet/pull/17255#issuecomment-572465096
 
 
   @marcoabreu 
   If you explicitly specified some dtype in an op with a `dtype` option, it means that you're not expecting to see whatever the default dtype is, isn't that a very natural thing? I'm not seeing any UX problem following this kind of logic.
   This new default type option is part of the new NumPy-compatible user experience for a closer compatibility with existing code written in NumPy, a lot of which pose the assumptions that array creation functions return arrays with float64 data type by default, and failure to meet such assumption could lead to crashes and unexpected numerical bugs. For a more smooth transition for those code to NP on MXNet, providing such an option instead of having to change all array creation functions in those code to explicitly say `dtype=np.float64` would, I assume, be providing a better UX.

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


With regards,
Apache Git Services