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 2019/10/04 02:46:02 UTC

[GitHub] [incubator-mxnet] igolan opened a new pull request #16373: Round and sign straight-through-estimators C operators.

igolan opened a new pull request #16373: Round and sign straight-through-estimators C operators.
URL: https://github.com/apache/incubator-mxnet/pull/16373
 
 
   ## Description ##
   Implemented sign and round straight-through-estimators operators in C.
   Straight-through-estimators have derivative of 1 everywhere instead of 0 everywhere, this is required for quantized training.
   
   ## Checklist ##
   ### Essentials ###
   - [x] Changes are complete (i.e. I finished coding on this PR)
   - [x] All changes have test coverage:
   - [x] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments are documented. 
   - For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
   - Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
   - [x] To the my best knowledge, examples are not affected by this change.
   
   ### Changes ###
   - [x] contrib.round_ste() including test and API doc
   - [x] contrib.sign_ste() including test and API doc
   
   ## Comments ##
   N/A

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


With regards,
Apache Git Services