jtkim-kaist / VAD

Voice activity detection (VAD) toolkit including DNN, bDNN, LSTM and ACAM based VAD. We also provide our directly recorded dataset.
834 stars 232 forks source link

Can Octave be used? #3

Open vladfulgeanu opened 6 years ago

vladfulgeanu commented 6 years ago

Is there a way in which Octave can be used instead of Matlab for running the main.m file? When I try to run it using Octave I get:

warning: called from
    main at line 5 column 1
warning: function ./lib/matlab/stft/example.m shadows a core library function
warning: called from
    main at line 5 column 1
MRCG extraction ...
ans = 1
ans = 1
warning: load: '/home/vlad/Downloads/VAD/lib/matlab/MRCG_features/f_af_bf_cf.mat' found by searching load path
warning: called from
    loudness at line 12 column 1
    gammatone at line 35 column 10
    MRCG_features at line 7 column 3
    mrcg_extract at line 26 column 14
    vad_func at line 7 column 30
    main at line 21 column 12
.........................
error: fftfilt: B must be a vector
error: execution exception in main.m
maitchison commented 6 years ago

I made a fork where I think I've fixed that by unrolling the loop which can be found here https://github.com/TheCacophonyProject/VAD I also made some small other changes to get it working on windows, and with stereo audio files.

jtkim-kaist commented 6 years ago

Rather than Octave, we have a plan to write our code only using the python. please wait for it thank you!

charlescc9 commented 6 years ago

A full Python implementation would be awesome!

maitchison commented 6 years ago

Python would be great! For the moment I submitted a PR with just the Octave change. It does slow things down a bit though, so I only run the Octave version if we are actually on Octave.

This is a great library by the way, really appreciate the work!

jtkim-kaist commented 6 years ago

Good news! a full python implementation is upcoming (to be updated until 2018/03/17)

shezanmirzan commented 6 years ago

When can we expect the python implementation?

jtkim-kaist commented 6 years ago

Sorry for the late update, we already have a fully python written test script however, the train script has been delayed. We will firstly update test script in this week sorry.

jtkim-kaist commented 6 years ago
wishvivek commented 4 years ago

Thanks for the test script. Could you also upload the train script soon?

Linus789 commented 4 years ago

Yeah, a train script not depending on matlab would be great.

sh: 1: matlab: not found