[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Discuss-gnuradio] Setting of initial values after start with FCD
From: |
Alexandru Csete |
Subject: |
Re: [Discuss-gnuradio] Setting of initial values after start with FCD |
Date: |
Thu, 19 Apr 2012 20:21:13 +0200 |
On Thu, Apr 19, 2012 at 7:56 PM, Johnathan Corgan
<address@hidden> wrote:
> On Wed, Apr 18, 2012 at 14:52, Alexandru Csete <address@hidden> wrote:
>
>>> This sounds like a call to the appropriate setter function is missing
>>> in the FCD source XML wrapper <make></make> section.
>>
>> The generated python code looks right but there is definitely
>> something strange going on. I will take a look at it in the coming
>> days but if somebody has an idea don't hold back ;-)
>
> I think the parameters to the call to set_iq_corr() in the
> <make></make> section are reversed from the actual callback. Can you
> test this:
>
> diff --git a/gr-fcd/grc/fcd_source_c.xml b/gr-fcd/grc/fcd_source_c.xml
> index 7c55239..1b83cba 100644
> --- a/gr-fcd/grc/fcd_source_c.xml
> +++ b/gr-fcd/grc/fcd_source_c.xml
> @@ -19,7 +19,7 @@ self.$(id).set_freq_corr($ppm)
> self.$(id).set_dc_corr($dci,$dcq)
> #end if
> #if ($iq_phase() != 0.0) or ($iq_gain() != 1.0)
> -self.$(id).set_iq_corr($iq_phase,$iq_gain)
> +self.$(id).set_iq_corr($iq_gain,$iq_phase)
> #end if
> self.$(id).set_freq($freq)
> </make>
>
Yes, that's the problem. I tested the modification and it works.
Alex