-
Notifications
You must be signed in to change notification settings - Fork 90
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
Does Manyears support less microphone? #5
Comments
I'm not a developer, but from what I know: no. In fact, it doesn't support more microphones either. It just supports eight microphones (no more, no less). I say this because, reviewing the code, in file manyears-C/dsplib/parameters.h, lines 125-126:
And in file manyears-C/Qt4GUI/coreThread.cpp, line 127:
right after this line there is a snippet of code to configure each microphone:
This snippet is repeated for each microphone, and there are literally 8 instances of this code. So, it appears that the number of microphones is definitely hardcoded. However, seeing this current implementation, and having access to the functions in in manyears-C/dsplib/Geometry/microphones.c:
it appears that it is definitely doable to modify the code such that the number of microphones (as well as their position) be able to be changed to whatever the user wants. However, at first glance, it would involve some tweaks in manyears-C/Qt4GUI/coreThread.cpp, some modifications in the configuration GUI (since this is also hardcoded), some additions into the save and load configuration functions, as well as some sane checks so that everything is loaded up correctly. From this point on, however, from what I've seen in the code, it should be able to handle different types of array geometries. |
It is actually possible to run ManyEars with less than 8 microphones without having to modify the code. For instance, the manyears_ros package contains a demo for the Kinect, which has 4 microphones. The trick is to set the gain to 0.0 for the unused microphones. See this configuration file for an example: https://github.com/introlab/irl_audio/blob/master/manyears_ros/data/kinect.mes |
Didn't think of that. Handy. |
Thanks for your answers @francoisferland and @balkce |
Does manyears support less than 8 microphone?
The text was updated successfully, but these errors were encountered: