Daniel Povey
Daniel Povey
Oh, wait, sorry... I've moved the main development to the 'shortcut' branch in the official Kaldi repo, this branch is dead now.
done in #7
Pegah, development is now in the 'chain' branch in the official kaldi repo. Could you please try to apply your changes to that one instead? There have not been many...
just saw this- thanks! Can you please make the PR to github.com/k2-fsa/k2/ instead though?
... also, I think you are using the modified CTC topology (topo2), which allows 2 successive repeats of the same phone without blank in between, which may not be a...
I merged Kangwei's fix as it seems to be a straightforward fix of an undefined value, if there are further bugs we can fix separately.
If this is a graph-building exercise of some kind it may be better to use the kaldifst project https://k2-fsa.github.io/kaldifst/ to do the graph-building, then if needed it can be converted...
Sometimes, using /dev/stdin as a pseudo-file can be a workaround for this type of situation.
There are quite a few changes not in the directory you are adding. You might want to remove those as they are potential barriers to merging it. If there's anything...
Yeah we probably committed the sin of relying on things that are indirectly included. @danijel3 is there any chance you could commit a fix? It looks like we are supposed...