Open GoogleCodeExporter opened 8 years ago
Additional info:
in mrcp_pocketsphinx.c this statement is always false:
if(recognizer->is_input_timer_on == TRUE)
Original comment by damiank...@gmail.com
on 13 Apr 2013 at 5:19
the recognizer->is_input_timer_on flag is always false, but, even if it were
set to true at the beginning of the call...
Theres another problem here. If the microphone is completed muted before
placing the call, this statement is executed once or twice anyways:
if(score != 0 && recognizer->is_input_timer_on)
{
recognizer->is_input_timer_on = FALSE;
}
Not sure what can be done here.... There are partial results even though the
call is completely muted before placing.
Original comment by damiank...@gmail.com
on 13 Apr 2013 at 5:27
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
[deleted comment]
Fixed....
Commenting the following:
/* reset input timer as we have partial match now */
// if(score != 0 && recognizer->is_input_timer_on)
// {
// recognizer->is_input_timer_on = FALSE;
// }
And adding this:
det_event = mpf_activity_detector_process(recognizer->detector,frame);
switch(det_event) {
case MPF_DETECTOR_EVENT_ACTIVITY:
++ if(recognizer->is_input_timer_on)
++ {
++ apt_log(APT_LOG_MARK,APT_PRIO_INFO,"Stopping input timers.....
"APT_SIDRES_FMT ,RECOGNIZER_SIDRES(recognizer));
++ recognizer->is_input_timer_on = FALSE;
++ }
apt_log(APT_LOG_MARK,APT_PRIO_INFO,"Detected Voice Activity "APT_SIDRES_FMT,RECOGNIZER_SIDRES(recognizer));
pocketsphinx_start_of_input(recognizer);
break;
case MPF_DETECTOR_EVENT_INACTIVITY:
apt_log(APT_LOG_MARK,APT_PRIO_INFO,"Detected Voice Inactivity "APT_SIDRES_FMT,
RECOGNIZER_SIDRES(recognizer));
pocketsphinx_end_of_input(recognizer,RECOGNIZER_COMPLETION_CAUSE_SUCCESS);
break;
default:
break;
}
For some reason, the pocketsphinx plugin gets partial results even if the
microphone is muted, however, even though pocketsphinx is getting partial
results, MRCP correctly detects voice activity, therefore, stopping the input
timers where voice is properly detected fixes the issue.
A patch is attached..
Original comment by damiank...@gmail.com
on 13 Apr 2013 at 5:57
Attachments:
Original comment by achalo...@gmail.com
on 16 Apr 2013 at 1:45
Original issue reported on code.google.com by
damiank...@gmail.com
on 13 Apr 2013 at 5:10