cfo detection

[Benjamin Menküc]

isnt in this line the conjugate needed?

    offset_radians = angle(dot(cyclic_prefix, symbol_tail)) / fft_size;

like

    offset_radians = angle(dot(cyclic_prefix, conj(symbol_tail))) / fft_size;

[Kobi Kai Calev]

No, the second argument in a complex dot project, is already conj() by definition AFAIK

[Benjamin Menküc]

thats also what I suspected, but I did not read anything about it in the matlab documentation.
But now I just tried it out in matlab and You are right

dot(1j,1j) = 1

I was confused because np.dot does not include conj and np usually tries to make the commands behave similar as in matlab.

[Kobi Kai Calev]

yes, this is always slightly confusion between libraries, btw: note that np.vdot() does the conj(), where indeed np.dot() does not.