|
From: | Robert McGwier |
Subject: | Re: [Discuss-gnuradio] Bug in gr_pll_carriertracking.cc |
Date: | Thu, 24 Nov 2011 23:52:39 -0500 |
The last time I did this I had written the routine ....
;-)
Bob
Heh, sometimes you have to e-mail a public list before any of it makes sense. Happens to me all the time =)Cheers,BenOn Wed, Nov 23, 2011 at 8:53 AM, Marcus M <address@hidden> wrote:
My bad. The phase_detector() function returns the difference. Ignore this message.On Wed, Nov 23, 2011 at 10:12 AM, Marcus M <address@hidden> wrote:
Hi,
I have the latest gnuradio release and I think the phase error calculation in gr_pll_carriertracking.cc might be wrong.
In Line 107 the phase error is calculated as,
error = phase_detector(iptr[i],d_phase);
whereas, I think it should be as shown below as the phase error is calculated from the signal that has been corrected in phase.
error = phase_detector(optr[i],d_phase);
Am I right?
Thanks
_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
[Prev in Thread] | Current Thread | [Next in Thread] |