[Home]

Summary:ASTERISK-00848: [patch] basic app_cepstral
Reporter:scaredycat (scaredycat)Labels:
Date Opened:2004-01-15 09:28:04.000-0600Date Closed:2011-06-07 14:05:22
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_cepstral_final.tar.gz
( 1) cepstral.tar.gz
Description:usage is something like:

exten => *388,1,setcallerid(${CALLERIDNUM})
exten => *388,2,AGI(lookup)
exten => *388,3,SetCallerID("${LOCATION}" <${CALLERIDNUM}>)
exten => *388,4,Answer
exten => *388,5,Wait(1)
exten => *388,6,Cepstral("Caller location is ${LOCATION}"|/tmp/${UNIQUEID})
exten => *388,7,wait(3)

needs tarting up and double checking but it works for me
Comments:By: John Todd (jtodd) 2004-01-15 09:31:08.000-0600

There are rumors of someone named "rob--" on the IRC channel who is working on an alternate app_cepstral that doesn't use temp files, but my belief is "working code wins the race" so until we see the alternate version I'd go with this one.  I haven't tested yet, but look forward to trying it out in my "spare time."

By: scaredycat (scaredycat) 2004-01-15 09:34:37.000-0600

I want to move the use of a tmp file to be internal... so you don't need to specify.. forgot to add that this creates the file, plays it back then removes it...

SC

By: scaredycat (scaredycat) 2004-01-15 11:35:01.000-0600

no longer requires path and filename at the end eg : |/tmp/${UNIQUEID} - uses /tmp/${UNIQUEID} internally...

enjoy

SC

By: scaredycat (scaredycat) 2004-01-15 11:36:04.000-0600

DOH! Use  app_cepstral_final.tar.gz ... (dunno how to delete old files)

By: Brian West (bkw918) 2004-01-15 14:19:09.000-0600

Now I have to buy cepstral now damn it... haha

By: Brian West (bkw918) 2004-01-27 18:30:23.000-0600

Do the header files come with the basic cepstral or do you have to buy the SDK to get them?

By: Brian West (bkw918) 2004-01-27 19:45:45.000-0600

This thing don't compile or load.

By: scaredycat (scaredycat) 2004-01-29 06:30:32.000-0600

bkw,

you just lost the right to ever complain if someone provides no information ina bug report. How about a trace.. something... anything... you know  - a little more info...

did you add:

app_cepstral.so: app_cepstral.c
     $(CC) -shared -Xlinker -x -o $@ $< -L/opt/theta/lib -ltheta -lm -ldl

in your makefile?

SC

edited on: 01-29-04 06:15

By: Brian West (bkw918) 2004-02-02 01:09:46.000-0600

haha I tried to talk to you on IRC.. but you wouldn't talk to me.. happen to know why you didn't include a diff for that? :P

By: Brian West (bkw918) 2004-02-02 01:25:59.000-0600

ok works great.

A few compile errors I have fixed those.

in /opt/theta/include/theta_support.h line 372 should be:

THETA_API cst_wave *new_wave(void);

Makefile entry is like this:

app_cepstral.so: app_cepstral.o
       $(CC) -shared -Xlinker -x -o $@ $< -L/opt/theta/lib -ltheta -lm -ldl

By: scaredycat (scaredycat) 2004-02-02 09:26:35.000-0600

>haha I tried to talk to you on IRC.. but you wouldn't talk to me.. happen to >know why you didn't include a diff for that? :P  

Hey! it was 3am and I did paste that in the pm window...  


FYI I didn't have to make any of your changes to theta_support.h ... but then I'm a professional :P

SC

By: Brian West (bkw918) 2004-02-02 23:49:52.000-0600

attached is a standalone ready for asterisk-addons. :P

Care if I work on this to add options such as removing the /tmp file when it done with it?  Add the noanswer option? and such?

bkw

By: Brian West (bkw918) 2004-02-03 01:08:27.000-0600

Ok added a noanswer option. fixed it so it can now delete the temp file it creats in /tmp.  I also have to get the uniqueid fixed thats bad when you run off the end of the var and its garbage.  :P

By: Brian West (bkw918) 2004-02-03 01:10:11.000-0600

Ooops forgot to remove my debug info. :P

By: scaredycat (scaredycat) 2004-02-03 02:20:39.000-0600

mmm... Interesting that it doesn;t delete the files on your system. It works fine on mine using the ast_filedelete function...

     strncpy(options,"\0",sizeof(options)-1);
     pbx_substitute_variables_helper(chan,"${UNIQUEID}",options, sizeof(options)-1);

!

Add whatever else you feel would be useful for the app...

SC

edited on: 02-03-04 02:21

By: Brian West (bkw918) 2004-02-03 09:30:17.000-0600

strncpy won't work because the strlen of UNIQUEID return is 29 (value + garbage)  so you are tacking the terminator to the end of that garbage.  I will investigate more.

ast_filedelete(nsfname,"wav"); // works

bkw

By: Brian West (bkw918) 2004-02-03 15:22:54.000-0600

accually you would strncpy NULL and then it would append the \0 wouldn't it?

By: Brian West (bkw918) 2004-02-03 19:32:52.000-0600

This will never go in CVS because the theta engine isn't GPL.