-
Notifications
You must be signed in to change notification settings - Fork 42
"Check failed" after decoding + shortest_path #31
Comments
As G contains disambiguation symbols bug L does not, we should use |
Wait, does G really contain disambiguation symbols? I thought they only exist in the L alphabet (as "phones") and not in the G alphabet (as "words"). |
You can check the end of |
@pzelasko here is the reason why we need #0 in G http://kaldi-asr.org/doc/graph.html#graph_disambig. @qindazhu thanks for your update, looking forward to your decoding example! |
Thanks for your comment!
I don't think we committed the fix yet (@qindazhu please do!) but we
discovered, I think, a problem in the script that creates G.fst in
snowfall, that it was including the end of sentence marker `</s>`.
…On Fri, Dec 4, 2020 at 12:42 PM Xiaohui Zhang ***@***.***> wrote:
@pzelasko <https://github.com/pzelasko> here is the reason why we need #0
in G http://kaldi-asr.org/doc/graph.html#graph_disambig. @qindazhu
<https://github.com/qindazhu> thanks for your update, looking forward to
your decoding example!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#31 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZFLOZG6HI6FJDIIL5KJBLSTBSDDANCNFSM4UFEQKZA>
.
|
Yeah, I have removed Anyway, will make a PR with my lastest updates which removing |
Fixed |
Thanks! Never too late to learn. |
I've been playing around with the recipe to get to know k2 better. I trained the model for 10 epochs and then tried to perform decoding with the following code (running everything on the CPU):
This crashes with the following message + stack trace:
Some questions:
run.sh
.The text was updated successfully, but these errors were encountered: