RealAntennas icon indicating copy to clipboard operation
RealAntennas copied to clipboard

Wrong Unit of N0 in the "Noise" Section of Debugger?

Open SaberCF opened this issue 2 years ago • 2 comments

I found that the unit of N0 in "Noise" section is dB/Hz, and I think it should be dBm/Hz, just like the N0 in the "Min Link Channel Noise Power" below.

https://github.com/DRVeyl/RealAntennas/blob/90fb5450105e6ebebfb3dfb620b4881181c9addb/src/RealAntennasProject/Network/ConnectionDebugger.cs#L111

SaberCF avatar Feb 07 '23 15:02 SaberCF

Well, the "Min Link Channel Noise Power" seems also have some faults. The two "dB" in {RATools.LogScale(data.minSymbolRate):F1} dB and {channelNoise:F1} dB" should be "dBm" I think. The unit of "Channel Noise power" in the "Min Link Eb/N0" below is "dBm". https://github.com/DRVeyl/RealAntennas/blob/90fb5450105e6ebebfb3dfb620b4881181c9addb/src/RealAntennasProject/Network/ConnectionDebugger.cs#L127 https://github.com/DRVeyl/RealAntennas/blob/90fb5450105e6ebebfb3dfb620b4881181c9addb/src/RealAntennasProject/Network/ConnectionDebugger.cs#L129

SaberCF avatar Feb 07 '23 16:02 SaberCF

Appears I was a little sloppy with units in crafting the debugger. Thanks for the dimensional analysis. I'll keep this open and make a change on next release... which has no timeline currently. [Maybe if Egg's Skopos gets some progress, he has some things he wants to PR, and I might return enough for another bugfix update and merge his things.]

You're correct, N0 will be dBm/Hz. Channel noise power should be dBm. Line 129 looks correct. Line 127 I will keep minSymbolRate in dB [it's just the log scale of the bandwidth] but channelNoise at the very end is dBm.

DRVeyl avatar Feb 12 '23 21:02 DRVeyl