Backport changes from AppTek #153
Conversation
|
This started because I was annoyed with gcc complaining about deprecated function usage that I already fixed in the AppTek codebase and then morphed quickly into let's backport as much as possible to reduce friction in the future. |
|
I just ran a test with a CTC in the AdvancedTreeSearch. I directly compared the result from the current version on master and this commit and unfortunately I'm observing a degradation from 7.26% to 7.39% WER. |
|
Once again it was the LM Lookahead. I'm now getting 7.26% now, but maybe you @larissakl could confirm that. Thanks! |
|
Ah, exactly the same loop again. At least this time it didn't take us a year to notice that 😄 |
There was a problem hiding this comment.
So I checked that the AdvancedTreeSearch and the TreeTimesyncBeamSearch produce the same output with one of my test setups and I had a manual look through some files I consider important and that looks good. However I cannot guarantee that other tools are not broken/affected.
Mostly whitespace changes, but also some refactoring / minor fixes.