respeaker / usb_4_mic_array

ReSpeaker 4 Mic Array with builtin VAD, DOA, AEC, Beamforming & NS
https://www.seeedstudio.com/ReSpeaker-Mic-Array-v2.0-p-3053.html
Apache License 2.0
141 stars 65 forks source link

Odas cfg is incorrect #8

Closed taospartan closed 6 years ago

taospartan commented 6 years ago

The microphone layout for use in odas is totally wrong. Results in false doa and separation.

xiongyihui commented 6 years ago

@taospartan Can you explain what is wrong?

taospartan commented 6 years ago

If you look at the microphone coordinates they are spaced incorrectly, looks like the mics are positioned in a straight line

Many thanks

On 9 Jul 2018, at 02:40, Yihui Xiong notifications@github.com wrote:

@taospartan Can you explain what is wrong?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

xiongyihui commented 6 years ago

They are not in a straight line.

        # Microphone 2
        { 
            mu = ( -0.032, +0.000, +0.000 ); 
            sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
            direction = ( +0.000, +0.000, +1.000 );
            angle = ( 80.0, 100.0 );
        },

        # Microphone 3
        { 
            mu = ( +0.000, -0.032, +0.000 ); 
            sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
            direction = ( +0.000, +0.000, +1.000 );
            angle = ( 80.0, 100.0 );
        },

        # Microphone 4
        { 
            mu = ( +0.032, +0.000, +0.000 ); 
            sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
            direction = ( +0.000, +0.000, +1.000 );
            angle = ( 80.0, 100.0 );
        },

        # Microphone 5
        { 
            mu = ( +0.000, +0.032, +0.000 ); 
            sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
            direction = ( +0.000, +0.000, +1.000 );
            angle = ( 80.0, 100.0 );        
        }
taospartan commented 6 years ago

My mistake, but they are still not plotting sound sources correctly, this is my config for the mics see what you think

mics = (

    # Microphone 2
    {
        mu = ( +0.0239, +0.0239, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000,

+0.000, +0.000, +0.000 ); direction = ( +0.000, +0.000, +1.000 ); angle = ( 90.0, 100.0 ); },

    # Microphone 3
    {
        mu = ( -0.0239, +0.0239, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000,

+0.000, +0.000, +0.000 ); direction = ( +0.000, +0.000, +1.000 ); angle = ( 90.0, 100.0 ); },

    # Microphone 4
    {
        mu = ( -0.0239, -0.0239, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000,

+0.000, +0.000, +0.000 ); direction = ( +0.000, +0.000, +1.000 ); angle = ( 90.0, 100.0 ); },

    # Microphone 5
    {
        mu = ( +0.0239, -0.0239, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000,

+0.000, +0.000, +0.000 ); direction = ( +0.000, +0.000, +1.000 ); angle = ( 90.0, 100.0 ); }

);

On 9 July 2018 at 09:51, Yihui Xiong notifications@github.com wrote:

They are not in a straight line.

    # Microphone 2
    {
        mu = ( -0.032, +0.000, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
        direction = ( +0.000, +0.000, +1.000 );
        angle = ( 80.0, 100.0 );
    },

    # Microphone 3
    {
        mu = ( +0.000, -0.032, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
        direction = ( +0.000, +0.000, +1.000 );
        angle = ( 80.0, 100.0 );
    },

    # Microphone 4
    {
        mu = ( +0.032, +0.000, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
        direction = ( +0.000, +0.000, +1.000 );
        angle = ( 80.0, 100.0 );
    },

    # Microphone 5
    {
        mu = ( +0.000, +0.032, +0.000 );
        sigma2 = ( +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000, +0.000 );
        direction = ( +0.000, +0.000, +1.000 );
        angle = ( 80.0, 100.0 );
    }

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/respeaker/usb_4_mic_array/issues/8#issuecomment-403407446, or mute the thread https://github.com/notifications/unsubscribe-auth/AYDyNgWqyubGHVBTOf6OC50BwnwIDx6xks5uExmDgaJpZM4VGUfL .

xiongyihui commented 6 years ago

It's just the coordinate rotates 45 degree.

taospartan commented 6 years ago

I understand but unless you do rotate your coords 45 degrees odas does not work as intended especially the tracking . Feel free to test your self

Sent from my iPhone

On 9 Jul 2018, at 15:25, Yihui Xiong notifications@github.com wrote:

It's just the coordinate rotates 45 degree.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

xiongyihui commented 6 years ago

I tried your configuration, but didn't see much difference. Maybe it is the difference between angle = ( 90.0, 100.0 ); and angle = ( 80.0, 100.0 );. Otherwise, it should be a bug.

taospartan commented 6 years ago

Ok

Sent from my iPad

On 10 Jul 2018, at 02:16, Yihui Xiong notifications@github.com wrote:

I tried your configuration, but didn't see much difference. Maybe it is the difference between angle = ( 90.0, 100.0 ); and angle = ( 80.0, 100.0 );. Otherwise, it should be a bug.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.