Re: [Discuss-gnuradio] reset() for probe_avg_mag_sqrd

[Martin Braun]

Stefan,

Johnathan asked you a couple of questions regarding your patch. I'm not
sure if you've already answered him, but the usual approach here is that
we continue patch-related discussions on the issue tracker. So what it
looks like from here is that's actually stalling on you :)

Once things are in the issue tracker, they don't get forgotten. However,
sometimes it takes longer to merge/address things. Hope this doesn't
discourage you or any other contributor.

Cheers,
M

On 08/21/2014 07:43 PM, Stefan Oltmanns wrote:
> Hi,
> I posted the issue with a patch about 2 weeks ago, I´m not sure anybody
> has seen it yet:
> http://gnuradio.org/redmine/issues/709
> Would be great if it could be included.
> 
> Thanks
> Stefan
> 
> 
> Am 04.08.2014 11:05, schrieb Martin Braun:
>> Stefan,
>>
>> this might be useful, can you post an issue on gnuradio.org, and
>> describe as clearly as possible what you need? A patch, even if small,
>> also helps to make things clearer for us.
>>
>> M
>>
>> On 08/02/2014 07:50 PM, Stefan Oltmanns wrote:
>>> Hello,
>>>
>>> my application is monitoring the RF level and often changeing the
>>> frequency. Unfortunately it takes some time for probe_avg_mag_sqrd to
>>> adjust to the new RF level (about 2 seconds if old level -10dB and new
>>> level -40dB).
>>> I found out the the iir-filter probe_avg_mag_sqrd uses has a reset
>>> function, so all I did was adding a reset-function in probe_avg_mag_sqrd
>>> to call it and it seems so work fine.
>>> Is there any reason probe_avg_mag_sqrd doesn´t have a reset function
>>> yet? Is there some problem I don´t see?
>>> In case not, could that be added in official GNURadio? Otherwise my
>>> application would only work with modified GNURadio (or I would need to
>>> include my modified probe_avg_mag_sqrd with a new name in my
>>> appliaction). I would prefer to have this feature in official GNURadio,
>>> as it´s probably useful for somebody else as well.
>>>
>>> Best regards
>>> Stefan
>>>
>>>
>>> Note: I´m not sending a patch because it´s just 3 lines of code and I
>>> don´t know Python, so somebody else would have to complete it anyway.
>>>
>>> _______________________________________________
>>> 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
>>
> 
> 
> _______________________________________________
> Discuss-gnuradio mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
>