leezu commented on issue #17051: fix parameter names in the estimator api
URL: https://github.com/apache/incubator-mxnet/pull/17051#issuecomment-565708404
 
 
   @sxjscience it is bad practice to release a breaking change to an API, with 
the change already being deprecated at the time of release. There are only two 
options IMO: 1) revert the change and stay with the old API 2) Fix the API 
before the release.
   
   Even if the feature is experimental, it's bad practice to change the API 
unnecessarily many times.
   
   In the future, let's go via the revert route. 

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