ZiyueHuang commented on issue #9848: add infer_type for regression ops, fix 
issues#9847
URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-367688993
 
 
   Unittest is written in symbolic mode while the issue is in imperative mode. 
In symbolic mode, infer_type pass has a default infer function `SameType` 
https://github.com/apache/incubator-mxnet/blob/master/src/executor/infer_graph_attr_pass.cc#L403-L408,
 but in imperative mode, there is no default infer type function 
https://github.com/apache/incubator-mxnet/blob/master/src/imperative/imperative_utils.h#L123.
 So this is an inconsistent behaviour between symbolic mode and imperative 
mode. Should we correct it? cc @reminisce @eric-haibin-lin 

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to