odas icon indicating copy to clipboard operation
odas copied to clipboard

Changing the nPairs during localization.

Open alexengbloom opened this issue 6 years ago • 4 comments

Hello @FrancoisGrondin ,

I am trying to change the number of pairs used for localization. It seems like ODAS is doing permutation and is using all possible pairs as possible.

If I simply try to modify the freq2freq.c (freq2freq_product_process) and try to hard code obj->products->nPairs to a smaller number I get a Bus error; memory problem. Changing the nPairs in the mic.c does the same.

It's quite difficult to know which part I should modify. I tracked the problem and it seems like the error is caused by line : thread_join(aobjs->asrc_hops_mics_object->thread); in function threads_multiple_join()

Could you help me solve the problem?

Thanks!

alexengbloom avatar Sep 07 '18 18:09 alexengbloom

Hi Alex,

Sorry for the late reply. You should normally change the number of microphones, and the number of pairs will be calculated automatically N = M(M-1)/2, where N is the number of pairs, and M is the number of microphones.

Out of curiosity, is there a particular reason why you'd need to change the number of pairs?

Thanks,

Francois

FrancoisGrondin avatar Sep 08 '18 00:09 FrancoisGrondin

Hi @FrancoisGrondin ,

Yeah. I saw the code doing N = M(M-1)/2.

I wanted to reduce the number of pairs to save some computation power; make the program more lightweight. I am planning to choose a number of pairs instead of doing N = M(M-1)/2; bu still using all microphones available.

Can you help fix this part? I tried but ended up with a lot of memory related error.

alexengbloom avatar Sep 08 '18 03:09 alexengbloom

Are there any updates on this issue? I would also be interested in this feature.

Do you (@FrancoisGrondin ) have any suggestions or ideas of how to achieve this in a simple manner? Did you (@alexengbloom ) make any progress?

luc2yj avatar Oct 22 '18 04:10 luc2yj

@luc2yj Not yet. @FrancoisGrondin Would you have a chance to look at this issue? Or provide some advice on how to modify the code to make it work with less nPairs?

alexengbloom avatar Oct 22 '18 17:10 alexengbloom