----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3563/#review11995 -----------------------------------------------------------
How do you foresee this being invoked for ARI? the existing channel variable stuff? /branches/12/funcs/func_talkdetect.c <https://reviewboard.asterisk.org/r/3563/#comment21911> It may be worthwhile adding some information about using this with the AGC and DENOISE dialplan functions. /branches/12/funcs/func_talkdetect.c <https://reviewboard.asterisk.org/r/3563/#comment21917> Er this reads weird. Rephrase. /branches/12/funcs/func_talkdetect.c <https://reviewboard.asterisk.org/r/3563/#comment21918> Is it possible for another thread to jump in here and do the same thing? /branches/12/funcs/func_talkdetect.c <https://reviewboard.asterisk.org/r/3563/#comment21919> Good job remembering this can happen. :D - Joshua Colp On May 25, 2014, 4:47 a.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3563/ > ----------------------------------------------------------- > > (Updated May 25, 2014, 4:47 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23786 > https://issues.asterisk.org/jira/browse/ASTERISK-23786 > > > Repository: Asterisk > > > Description > ------- > > This patch adds a new channel function TALK_DETECT that, when set on a > channel, causes events indicating the start/stop of talking on a channel to > be emitted to both AMI and ARI clients. Why a channel function? Because both > ARI and AMI can set it on the channel, making it (slightly) more useful than > existing speech/silence detection dialplan applications. > > The function allows setting both the silence threshold (the length of silence > after which we decide no one is talking) as well as the talking threshold > (the amount of energy that counts as talking): > > same => n,Set(TALK_DETECT(set)=2500,256) > > Parameters can be updated on a channel: > > same => n,Set(TALK_DETECT(set)=) ; Enable talk detection > same => n,Set(TALK_DETECT(set)=,128) ; Set the talking threshold on the > existing talk detection to 128 > > And talk detection can be removed: > > same => n,Set(TALK_DETECT(remove)=) > > The events raised by the function use a nomenclature similar to existing > AMI/ARI events. > For AMI: ChannelTalkingStart/ChannelTalkingStop > For ARI: ChannelTalkingStarted/ChannelTalkingFinished > > Both 'end' events include the duration of talking that was detected (in > milliseconds). > > > Diffs > ----- > > /branches/12/rest-api/api-docs/events.json 414470 > /branches/12/res/ari/ari_model_validators.c 414470 > /branches/12/res/ari/ari_model_validators.h 414470 > /branches/12/main/stasis_channels.c 414470 > /branches/12/main/audiohook.c 414470 > /branches/12/include/asterisk/stasis_channels.h 414470 > /branches/12/funcs/func_talkdetect.c PRE-CREATION > > Diff: https://reviewboard.asterisk.org/r/3563/diff/ > > > Testing > ------- > > See review https://reviewboard.asterisk.org/r/3564 > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev