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
