[GitHub] szha commented on issue #7727: change contrib CTC to zero-indexed, in log space. sequence mask for grad

[gi...@git.apache.org]

szha commented on issue #7727: change contrib CTC to zero-indexed, in log space. sequence mask for grad
URL: https://github.com/apache/incubator-mxnet/pull/7727#issuecomment-327629133
 
 
   Discussed with @piiswrong offline. I will make the following changes:
   1. get rid of the `padding_mask` argument that I added in my previous PR
   2. make the default behavior of contrib CTC 1-indexed to maintain backward compatibility.
   3. add a boolean blag to specify whether label is 0-indexed or 1-indexed. Use padding mask -1 for 0-indexed and padding mask 0 for 1-indexed for backward compatibility.
   4. in gluon, remove padding_mask argument, and always use 0-indexed label.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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