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:
[email protected]


With regards,
Apache Git Services

Reply via email to