You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/07/13 17:55:27 UTC

[GitHub] [incubator-mxnet] ptrendx commented on issue #18697: AMP for mx2

ptrendx commented on issue #18697:
URL: https://github.com/apache/incubator-mxnet/issues/18697#issuecomment-657703538


   Hi @szha, could you clarify how this approach (graph pass) would work for the non-hybridized execution? I don't think it would be applied then, which would produce a difference in the behavior. That would negatively impact debuggability of models for example, which is pretty bad.
   
   That said, one of the prerequisites for the graph pass is to remove AMP lists and instead put their content as op's attributes and I believe this should be doable and beneficial.


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