-
Notifications
You must be signed in to change notification settings - Fork 17
v4.0: Changed parts of functions fftconv2 and gfbank_calcaxis #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…peed-up of 1.4 on tested matrices
Thank you for your contribution! Could you provide the matrices or their size and number? Did you benchmark the code with Matlab (which version) or Octave (which version and BLAS library)? Best regards, |
Hi Marc-René, My pleasure! I tested yesterday on a few matrices not output by log_el_spectrogram, so I just did test this. Tested with num_bands=31, 62 and 23, code v3.0 and v.4.0 (6 different colours), and a matrix logmelspec initially of size 31x408 (when the waveform 'signal' is of length 65639). I vary the length of the signal to be in [500 1000 5000 10000 50000 100000 500000]. Code used: [signal, fs] = audioread(listWavsTIMIT{kk});
end figure ; Version :MATLAB Version: 9.0.0.341360 (R2016a) Java Version: Java 1.7.0_75-b13 with Oracle Corporation Java HotSpot(TM) 64-Bit Server VM mixed modeResult: and
ans =
So almost systematically v4 is faster, but less than what I observed yesterday. By the way, I’m using your code to compare results on the TIMIT dataset. If the results are of interest to you, we can discuss it on Skype. I’m Alban, from the IHR in Nottingham we met last year in Groningen. Cheers,
|
Hi Alban, thank you for your report! I appreciate that you use and try to improve the code. This repository contains a reference implementation and mainly targets educational and academic use (at the cost of performance). However, if you are interested in a high-performance version of the (S)GBFB feature extraction, we could think about a performance-optimized implementation in Matlab/Octave or an implementation in C. I don't have the resources for that at the moment because the performance of the feature extraction has never really been a bottleneck in my experiments. By the way, the separable Gabor filter bank (SGBFB) processing is more efficient than the GBFB processing if it is used with, e.g., I am always interested in published results that use GBFB or, even better, SGBFB features. Best, |
Hi Marc, I appreciate your code is meant to remain clean. In the previous In the beginning, I wasn’t using the SGBFB function much since in my Thank you for your offer to discuss the results. At the moment, I Best, — Alban On Sep 19, 2016, at 09:51, Marc René Schädler notifications@github.com wrote: Hi Alban, thank you for your report! I appreciate that you use and try to This repository contains a reference implementation and mainly targets I want to keep the code easily understandable, and wont accept changes However, if you are interested in a high-performance version of the By the way, the separable Gabor filter bank (SGBFB) processing is more I am always interested in published results that use GBFB or, even If you would like feedback on some unpublished results, you can send Best, Marc — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread. |
For a speed-up of about 1.4 on tested matrices