szha commented on a change in pull request #7727: change contrib CTC to zero-indexed, in log space. sequence mask for grad URL: https://github.com/apache/incubator-mxnet/pull/7727#discussion_r137405194
########## File path: src/operator/contrib/ctc_loss.cc ########## @@ -35,7 +35,7 @@ ctcStatus_t compute_ctc_cost(const Tensor<cpu, 3, DType> activations, void *workspace, int train) { int minibatch = static_cast<int>(activations.size(1)); int alphabet_size = static_cast<int>(activations.size(2)); - int blank_label = 0; + int blank_label = alphabet_size-1; Review comment: Thanks, @zhiheng-huang. I will make an example like tf doc did to clarify. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services