View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005820 | ardour | bugs | public | 2014-01-11 14:00 | 2014-03-12 16:22 |
Reporter | piem | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | new | Resolution | open | ||
Summary | 0005820: patch to update to aubio 0.4.0 | ||||
Description | hi, here is a quick patch to update to the latest release of aubio. the patch should apply against 3.5.143. thanks, Paul | ||||
Tags | No tags attached. | ||||
2014-01-11 14:00
|
aubio.patch (8,151 bytes)
Description: merge latest vamp-aubio-plugins version to use aubio 0.4.0 Update libs/vamp-plugins/Onset.{cpp,h} to new aubio. Merge with the latest vamp-aubio-plugins revision 798ef8d. See http://git.aubio.org/?p=vamp-aubio-plugins.git;a=summary. Author: Paul Brossier <piem@debian.org> Bug-Debian: http://bugs.debian.org/733969 Last-Update: 2014-12-29 --- a/libs/vamp-plugins/Onset.cpp +++ b/libs/vamp-plugins/Onset.cpp @@ -22,29 +22,32 @@ using std::cerr; using std::endl; +const char *getAubioNameForOnsetType(OnsetType t) +{ + // In the same order as the enum elements in the header + static const char *const names[] = { + "energy", "specdiff", "hfc", "complex", "phase", "kl", "mkl", "specflux" + }; + return names[(int)t]; +} + Onset::Onset(float inputSampleRate) : Plugin(inputSampleRate), m_ibuf(0), - m_fftgrain(0), m_onset(0), - m_pv(0), - m_peakpick(0), m_onsetdet(0), - m_onsettype(aubio_onset_complex), + m_onsettype(OnsetComplex), m_threshold(0.3), - m_silence(-90), - m_channelCount(1) + m_silence(-70), + m_minioi(4) { } Onset::~Onset() { - if (m_onsetdet) aubio_onsetdetection_free(m_onsetdet); + if (m_onsetdet) del_aubio_onset(m_onsetdet); if (m_ibuf) del_fvec(m_ibuf); if (m_onset) del_fvec(m_onset); - if (m_fftgrain) del_cvec(m_fftgrain); - if (m_pv) del_aubio_pvoc(m_pv); - if (m_peakpick) del_aubio_peakpicker(m_peakpick); } string @@ -74,7 +77,7 @@ int Onset::getPluginVersion() const { - return 1; + return 2; } string @@ -86,22 +89,18 @@ bool Onset::initialise(size_t channels, size_t stepSize, size_t blockSize) { - m_channelCount = channels; + if (channels != 1) { + std::cerr << "Onset::initialise: channels must be 1" << std::endl; + return false; + } + m_stepSize = stepSize; m_blockSize = blockSize; - m_ibuf = new_fvec(stepSize, channels); - m_onset = new_fvec(1, channels); - m_fftgrain = new_cvec(blockSize, channels); - m_pv = new_aubio_pvoc(blockSize, stepSize, channels); - m_peakpick = new_aubio_peakpicker(m_threshold); - - m_onsetdet = new_aubio_onsetdetection(m_onsettype, blockSize, channels); - - m_delay = Vamp::RealTime::frame2RealTime(4 * stepSize, - lrintf(m_inputSampleRate)); + m_ibuf = new_fvec(stepSize); + m_onset = new_fvec(1); - m_lastOnset = Vamp::RealTime::zeroTime - m_delay - m_delay; + reset(); return true; } @@ -109,6 +108,22 @@ void Onset::reset() { + if (m_onsetdet) del_aubio_onset(m_onsetdet); + + m_onsetdet = new_aubio_onset + (const_cast<char *>(getAubioNameForOnsetType(m_onsettype)), + m_blockSize, + m_stepSize, + lrintf(m_inputSampleRate)); + + aubio_onset_set_threshold(m_onsetdet, m_threshold); + aubio_onset_set_silence(m_onsetdet, m_silence); + aubio_onset_set_minioi(m_onsetdet, m_minioi); + + m_delay = Vamp::RealTime::frame2RealTime(4 * m_stepSize, + lrintf(m_inputSampleRate)); + + m_lastOnset = Vamp::RealTime::zeroTime - m_delay - m_delay; } size_t @@ -132,8 +147,8 @@ desc.identifier = "onsettype"; desc.name = "Onset Detection Function Type"; desc.minValue = 0; - desc.maxValue = 6; - desc.defaultValue = (int)aubio_onset_complex; + desc.maxValue = 7; + desc.defaultValue = (int)OnsetComplex; desc.isQuantized = true; desc.quantizeStep = 1; desc.valueNames.push_back("Energy Based"); @@ -143,6 +158,7 @@ desc.valueNames.push_back("Phase Deviation"); desc.valueNames.push_back("Kullback-Liebler"); desc.valueNames.push_back("Modified Kullback-Liebler"); + desc.valueNames.push_back("Spectral Flux"); list.push_back(desc); desc = ParameterDescriptor(); @@ -159,11 +175,22 @@ desc.name = "Silence Threshold"; desc.minValue = -120; desc.maxValue = 0; - desc.defaultValue = -90; + desc.defaultValue = -70; desc.unit = "dB"; desc.isQuantized = false; list.push_back(desc); + desc = ParameterDescriptor(); + desc.identifier = "minioi"; + desc.name = "Minimum Inter-Onset Interval"; + desc.minValue = 0; + desc.maxValue = 40; + desc.defaultValue = 4; + desc.unit = "ms"; + desc.isQuantized = true; + desc.quantizeStep = 1; + list.push_back(desc); + return list; } @@ -176,6 +203,8 @@ return m_threshold; } else if (param == "silencethreshold") { return m_silence; + } else if (param == "minioi") { + return m_minioi; } else { return 0.0; } @@ -186,18 +215,21 @@ { if (param == "onsettype") { switch (lrintf(value)) { - case 0: m_onsettype = aubio_onset_energy; break; - case 1: m_onsettype = aubio_onset_specdiff; break; - case 2: m_onsettype = aubio_onset_hfc; break; - case 3: m_onsettype = aubio_onset_complex; break; - case 4: m_onsettype = aubio_onset_phase; break; - case 5: m_onsettype = aubio_onset_kl; break; - case 6: m_onsettype = aubio_onset_mkl; break; + case 0: m_onsettype = OnsetEnergy; break; + case 1: m_onsettype = OnsetSpecDiff; break; + case 2: m_onsettype = OnsetHFC; break; + case 3: m_onsettype = OnsetComplex; break; + case 4: m_onsettype = OnsetPhase; break; + case 5: m_onsettype = OnsetKL; break; + case 6: m_onsettype = OnsetMKL; break; + case 7: m_onsettype = OnsetSpecFlux; break; } } else if (param == "peakpickthreshold") { m_threshold = value; } else if (param == "silencethreshold") { m_silence = value; + } else if (param == "minioi") { + m_minioi = value; } } @@ -216,17 +248,6 @@ d.sampleRate = 0; list.push_back(d); - d = OutputDescriptor(); - d.identifier = "detectionfunction"; - d.name = "Onset Detection Function"; - d.unit = ""; - d.hasFixedBinCount = true; - d.binCount = m_channelCount; - d.hasKnownExtents = false; - d.isQuantized = false; - d.sampleType = OutputDescriptor::OneSamplePerStep; - list.push_back(d); - return list; } @@ -235,21 +256,12 @@ Vamp::RealTime timestamp) { for (size_t i = 0; i < m_stepSize; ++i) { - for (size_t j = 0; j < m_channelCount; ++j) { - fvec_write_sample(m_ibuf, inputBuffers[j][i], j, i); - } + fvec_set_sample(m_ibuf, inputBuffers[0][i], i); } - aubio_pvoc_do(m_pv, m_ibuf, m_fftgrain); - aubio_onsetdetection(m_onsetdet, m_fftgrain, m_onset); + aubio_onset_do(m_onsetdet, m_ibuf, m_onset); - bool isonset = aubio_peakpick_pimrt(m_onset, m_peakpick); - - if (isonset) { - if (aubio_silence_detection(m_ibuf, m_silence)) { - isonset = false; - } - } + bool isonset = m_onset->data[0]; FeatureSet returnFeatures; @@ -263,11 +275,6 @@ m_lastOnset = timestamp; } } - Feature feature; - for (size_t j = 0; j < m_channelCount; ++j) { - feature.values.push_back(m_onset->data[j][0]); - } - returnFeatures[1].push_back(feature); return returnFeatures; } --- a/libs/vamp-plugins/Onset.h +++ b/libs/vamp-plugins/Onset.h @@ -20,6 +20,17 @@ #include <vamp-sdk/Plugin.h> #include <aubio/aubio.h> +enum OnsetType { + OnsetEnergy, + OnsetSpecDiff, + OnsetHFC, + OnsetComplex, + OnsetPhase, + OnsetKL, + OnsetMKL, + OnsetSpecFlux // new in 0.4! +}; + class Onset : public Vamp::Plugin { public: @@ -54,17 +65,14 @@ protected: fvec_t *m_ibuf; - cvec_t *m_fftgrain; fvec_t *m_onset; - aubio_pvoc_t *m_pv; - aubio_pickpeak_t *m_peakpick; - aubio_onsetdetection_t *m_onsetdet; - aubio_onsetdetection_type m_onsettype; + aubio_onset_t *m_onsetdet; + OnsetType m_onsettype; float m_threshold; float m_silence; + float m_minioi; size_t m_stepSize; size_t m_blockSize; - size_t m_channelCount; Vamp::RealTime m_delay; Vamp::RealTime m_lastOnset; }; |
|
fixed in 3.5-228-g1eaa30b -- #ifdef'ed to work with both aubio 3 + 4 Note: aubio4's vamp onset-plugin introduces a regression: it's 1 channel only. Hence I leave this bug is left open for now. the Rhythm Ferret does not yet allow explicit channel selection. |
|
This is definitely a regression from the ardour perspective |
|
what should be the correct behavior? mix down the signal to mono before analysis, or analyze each channel separately? |
|
hi, any news on this? thanks, Paul |
|
the behaviour should be either whatever aubio 3 did, or chosen at runtime. |
Date Modified | Username | Field | Change |
---|---|---|---|
2014-01-11 14:00 | piem | New Issue | |
2014-01-11 14:00 | piem | File Added: aubio.patch | |
2014-01-12 17:27 | x42 | Note Added: 0015568 | |
2014-01-14 22:42 | paul | Note Added: 0015575 | |
2014-02-06 15:29 | piem | Note Added: 0015656 | |
2014-03-12 15:40 | piem | Note Added: 0015711 | |
2014-03-12 16:22 | paul | Note Added: 0015712 |