Author Topic: vamp-libxtract too many outputs from plugin  (Read 7567 times)

beam

  • Newbie
  • *
  • Posts: 3
    • View Profile
vamp-libxtract too many outputs from plugin
« on: November 05, 2008, 16:48:24 »
Hi guys,

I'm trying to use vamp-libxtract plugins. When used with the vamp-simple-host, I get warning messages like this:

> ./vamp-simple-host vamp-libxtract:mfcc:mfcc test_mono.wav
WARNING: PluginAdapterBase::Impl::convertFeatures: Too many outputs from plugin (2, only should be 1)
...

output has only timestamp and no mfcc value.

I'm using latest stable releases :
- vamp SDK v1.3
- vamp-libxtract.so : 0.4.2.20071019
on linux 32bits (debian etch).

it happens with almost all libxtract plugins, but it works well with other plugin libraries
I tried recompiling vamp-libxtract from sources, but I got same results.

maybe i'm missing something ...

do you have an idea ?

++
beam

cannam

  • Administrator
  • Sr. Member
  • *****
  • Posts: 273
    • View Profile
Re: vamp-libxtract too many outputs from plugin
« Reply #1 on: November 06, 2008, 11:34:38 »
Yes, I can confirm this.  I'll look into it.


Chris

cannam

  • Administrator
  • Sr. Member
  • *****
  • Posts: 273
    • View Profile
Re: vamp-libxtract too many outputs from plugin
« Reply #2 on: November 06, 2008, 11:48:29 »
Yes, that was a stupid one.  Thanks for reporting it!

I've fixed it in Subversion (rev 907 at https://vamp.svn.sourceforge.net/svnroot/vamp/vamp-libxtract-plugins/trunk).  If you want to fix your local copy without retrieving a new one, this is the patch:

Code: [Select]
--- plugins/XTractPlugin.cpp (revision 906)
+++ plugins/XTractPlugin.cpp (working copy)
@@ -225,6 +225,7 @@
      m_outputBinCount = 1; break;
     }
 
+    m_outputDescriptors.clear();
     setupOutputDescriptors();
 
     m_initialised = true;

Of course, I will also push out a release with this fix soon.


Chris

beam

  • Newbie
  • *
  • Posts: 3
    • View Profile
Re: vamp-libxtract too many outputs from plugin
« Reply #3 on: November 06, 2008, 17:22:44 »
Thanks for the patch, everythings is ok now :-)

++
beam