On Thu, Apr 19, 2012 at 7:56 PM, Johnathan Corgan <[email protected]> wrote: > On Wed, Apr 18, 2012 at 14:52, Alexandru Csete <[email protected]> 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 _______________________________________________ Discuss-gnuradio mailing list [email protected] https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
