cmusphinx / sphinxtrain

Acoustic model trainer for CMU Sphinx
Other
178 stars 112 forks source link

Output an error message when we cannot execute a tool #10

Closed cshung closed 3 years ago

cshung commented 7 years ago

This change adds an error message in the perl script when we have problem executing a tool.

I don't have pocketsphinx installed, so when I run sphinxtrain, it gives me an odd error message saying the match file is not found. While that's true, it is hard for me to understand what's gone wrong. Turn out we used to just ignore the error of not being able to run pocketsphinx, so we don't have the match files.

With the change, the HTML file will indicate clearly cannot run pocketsphinx, hopefully that can save others time debugging this.

nshmyrev commented 7 years ago

Thanks for the patch @cshung, actually my dream is to get rid of html output altogether, it's enough to print things to stdout.

cshung commented 7 years ago

The message I added will go to STDOUT. I mention about the HTML just because the HTML will grep for the "ERROR" prefix and also appear there.

cshung commented 7 years ago

Added an extra change to fix a typo in the code!