adhearsion / adhearsion-asr

Adds speech recognition support to Adhearsion as a plugin.
http://adhearsion.com
MIT License
6 stars 6 forks source link

Added debug logging for ASR matches #3

Closed wdrexler closed 11 years ago

wdrexler commented 11 years ago

This is meant to allow match information to be displayed in debug logging mode, as per request by @bklang

benlangfeld commented 11 years ago

So, all of the info you're manually searching for is available via punchblock and the RubySpeech NLSML parser. Please use those. I will not merge this as is. Please also log the Result object that is built, and include confidence there.

benlangfeld commented 11 years ago

Can you please add the confidence to the Result and log from that object, instead of adding a special case conditional?

wdrexler commented 11 years ago

Does 54facea5b6771e088032a5d982bcb72de54647a9 look better?

benlangfeld commented 11 years ago

Much better. Any value in logging alternative completion reasons (reason.name), do you think?

wdrexler commented 11 years ago

There probably is. I haven't really dealt with Punchblock at all before, so I'm kind of learning slowly as I go (sorry).

benlangfeld commented 11 years ago

@bklang Comments?

@wdrexler I'd like to get logging of alternative completion reasons in before I merge this.

wdrexler commented 11 years ago

Yeah, I wrote that comment just before I left the office. Working on logging alternate reasons now.