$KALDI_ROOT/src/lmbin is missing in path.sh in some of the examples

[Andrey Simanovsky]

Hi,

It seems that after arpa rewrite $KALDI_ROOT/src/lmbin was added only to some of the path.sh files in the examples. For instance, it is not present in gale_arabic, gale_mandarin, and fisher_english. As a result, fisher_english script breaks after it fails to find arpa2fst.

With best regards,

Andrey Simanovsky

[Jan Trmal]

Thanks for reporting this.  I'm working on refactoring the way how path.sh works,  so I'll fix it soon.  Thanks.
Y.

--
You received this message because you are subscribed to the Google Groups "kaldi-help" group.
To unsubscribe from this group and stop receiving emails from it, send an email to kaldi-help+...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Daniel Povey]

Yenda, I think we should patch the recipes where it's missing if the bigger fix will take more than a day or so.

Dan

[Jan Trmal]

Ok,  I'll do it tomorrow. Y

[Jan Trmal]

@danpovey I propose a this fix:

Instead of doing all the changes we were discussing and instead of fixing the faulty path.sh "the old way", I could create the 

[kaldi]/src/path.sh and then source that file in every path.sh including cleaning up the recipe's local PATH so that it does not include the components $KALDI_ROOT/src/*, but it would still include the components from $KALDI_ROOT/tools

This is manageable fairly quickly and it's halfway in our plans. What do you think?

The gale_mandarin, for example, would look like this after modifications:

export KALDI_ROOT=$(pwd)/../../..

export PATH=$PWD/utils/:$KALDI_ROOT/tools/openfst/bin:$PWD:$PATH

. $KALDI_ROOT/src/path.sh

export LC_ALL=C

y.

[Daniel Povey]

makes sense.

[Daniel Povey]

OK, the issue should be fixed now.

Dan