Re: [Discuss-gnuradio] Setting of initial values after start with FCD

[Alexandru Csete]

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