eric-haibin-lin commented on a change in pull request #14885: [Fit-API] Adress 
PR comments
URL: https://github.com/apache/incubator-mxnet/pull/14885#discussion_r282140389
 
 

 ##########
 File path: python/mxnet/gluon/contrib/estimator/event_handler.py
 ##########
 @@ -263,108 +268,171 @@ def epoch_end(self, estimator, *args, **kwargs):
 
 
 class CheckpointHandler(BatchEnd, EpochEnd):
-    """Save the model after every epoch.
+    """Save the model after user define period
 
-    :py:class:`CheckpointHandler` save the network parameters every epoch
+    :py:class:`CheckpointHandler` saves the network architecture after first 
batch if the model
+    can be hybridized(), saves model parameters and trainer states after user 
defined period,
 
 Review comment:
   can be hybridized() -> can be fully hybridized. 
   `hybridized()` is not a function

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