eric-haibin-lin commented on pull request #18525:
URL: https://github.com/apache/incubator-mxnet/pull/18525#issuecomment-646755466


   @RuRo my apologies. I didn't realize that it just uses Module API to get the 
shape. It probably should just use `Symbol.infer_shape` API instead. Let me 
create an issue to track this and get back to this issue later once I'm done 
with my current task. 
   Regarding the `test_models` test - I am afraid that there will be more work 
to rewrite all tests without using Module, which will be deprecated in mxnet 
2.0. And in fact, there are many tests that require such rewrites and help from 
the community. Let me create a github issue to track all features impacted, and 
call for contributions. It will also be great if you have the bandwidth to help


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


Reply via email to