diff options
| author | Tom Rondeau <tom@trondeau.com> | 2014-12-02 15:11:05 -0500 |
|---|---|---|
| committer | Tom Rondeau <tom@trondeau.com> | 2014-12-03 19:02:26 -0500 |
| commit | 57ee0e7e122668a2b7f628ecb84fe1f90f9d2d11 (patch) | |
| tree | 4a2bee2978248f55c9f3c14cdb68d0bb953950ae /gr-digital/python/digital/qam_constellations.py | |
| parent | 92ac02f7fd7289d11fa6523e0491c6a6d8c3cb56 (diff) | |
digital: fixes issues with the constellation soft decoder, specifically how the decisions are calculated in the C++ code and some issues with the QAM16 constellation in particular.
This addresses issue #737. The patch attached to that issue is not actually valid and is only an ordering problem/confusion. I will be adding an example GRC flowgraph that compares the output of the hard decision and soft decision versions to the original input stream to show how they match.
Increased testing coverage in the QA to test certain known points as well as random samples.
Diffstat (limited to 'gr-digital/python/digital/qam_constellations.py')
| -rwxr-xr-x | gr-digital/python/digital/qam_constellations.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gr-digital/python/digital/qam_constellations.py b/gr-digital/python/digital/qam_constellations.py index 6f9f6bfab5..ed86d7e1e3 100755 --- a/gr-digital/python/digital/qam_constellations.py +++ b/gr-digital/python/digital/qam_constellations.py @@ -270,7 +270,7 @@ def sd_qam_16_0x0_0_1_2_3(x, Es=1): b2 = -abs(x_re) + boundary b0 = -abs(x_im) + boundary - return [(Es/2)*b3, (Es/2)*b2, (Es/2)*b1, (Es/2)*b0] + return [(Es/2.0)*b3, (Es/2.0)*b2, (Es/2.0)*b1, (Es/2.0)*b0] sd_qam_16 = sd_qam_16_0x0_0_1_2_3 sd_qam_16_0 = sd_qam_16 |