[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Discuss-gnuradio] CRC32 discrepancy
From: |
Andrej Rode |
Subject: |
Re: [Discuss-gnuradio] CRC32 discrepancy |
Date: |
Tue, 31 Oct 2017 21:32:27 +0100 |
User-agent: |
Mutt/1.7.2 (2016-11-26) |
Hi Eric & Marcus,
I just submitted a fix for this issue [0].
This should new have the correct behaviour.
Cheers
Andrej
[0] https://github.com/gnuradio/gnuradio/pull/1504
On Fri, Aug 04, 2017 at 11:32:13AM +0200, Marcus Müller wrote:
> Hi Eric!
>
> Thanks for the bug report! Tracking that as an issue on github [1] now.
>
> You don't happen to have a fix that you've tested already?
>
> Best regards,
>
> Marcus
>
>
> [1] https://github.com/gnuradio/gnuradio/issues/1407
>
>
> On 08/03/2017 04:54 PM, ERIC BANWART wrote:
> >
> > I am using the Stream CRC32 block and was having trouble getting the
> > CRCs to pass. After some debugging I discovered that the
> > calculated CRC is different depending on whether or not the source
> > data is packed. This can be observed using the attached flow graph.
> > The result of the packed stream appears to be correct. I looked at the
> > code for the block (crc32_impl_bb.cc) and that see that regardless of
> > the Packed setting in the block the CRC calculated with:
> > d_crc_impl.process_bytes(in, packet_length);
> >
> >
> > I believe that for the unpacked case, either the bits need be repacked
> > before the calculation or repeated calls to process_bits should be
> > used in place of the the call to process_bytes.
> >
> >
> > My version of gnuradio is: 3.7.12git-126-g37d373ac
> >
> >
> > Thanks!
> >
> > Eric
> >
> >
> >
> > _______________________________________________
> > 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
--
Andrej Rode
GPG Key: 750B CBBB 4A75 811A 4D5F 03ED 5C23 7FB8 9A7D A2AA
signature.asc
Description: Digital signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Discuss-gnuradio] CRC32 discrepancy,
Andrej Rode <=