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: [email protected]
With regards, Apache Git Services
