|Subject:||Re: [Discuss-gnuradio] Bug in freq_xlating_fir_filter_XXX|
|Date:||Wed, 9 Oct 2013 10:45:08 -0400|
On Tue, Oct 8, 2013 at 9:39 PM, Achilleas AnastasopoulosYes, please submit a patch for this. The taps are reversed inside the
> I was playing around with
> and noticed that the two do not give the same output
> for the same input (and center_freq=0 in the xlating filter).
> Looking at the implementation of the latter
> it is obvious why: the taps are reversed in the line:
> std::reverse(ctaps.begin(), ctaps.end());
> For consistency the taps should not be reversed (as in all other filters)
> We also need to set
fir filters, so this is redundant and confusing. Most people probably
use symmetric filter taps, which is why it has not been found.
Don't change the sign of the frequency. I know this is controversial,
> float fwT0 = 2 * M_PI * d_center_freq / d_sampling_freq;
> (instead of the minus sign in the code).
> unless there is an objection, I will go ahead and push a correction,
but from my experience with users, more people find the current way
easier to understand. We're telling the filter what the center
frequency is, which means that we will take a signal at Fc and
downshift it to DC. To me, if we're on carrier Fc and we specify -Fc
as the "Center Frequency", that's more confusing.
|[Prev in Thread]||Current Thread||[Next in Thread]|