Re: [OctDev] new release?

[Quentin S. <qsp...@ie...>]

David,

Thanks. The updates fixed hammgen. However, now I've found another bug. 
I don't know if it was there all along or newly introduced. The 
following command should produce the 16x255 parity check matrix for a 
BCH 2-error correcting code:
cyclgen(255,bchpoly(255,239))

The result instead is:
error: cyclgen: generator polynomial does not produce cyclic code

I verified the resulting polynomial against MATLAB and they matched.

regards,
Quentin

David Bateman wrote:

>Quentin,
>
>I wrote the hammgen function over a year ago now and I'm scratching
>my head to think why I wrote the test for an irreducible polynomial
>the way I did in cyclgen.cc and cyclpoly.cc. I vaguely remember that
>there was a reason, but not the details. I think it might have been 
>a limitation elsewhere that I later removed and have now fixed. In
>any case hammgen upto 16 seems to work correctly and a quick test
>of cyclgen and cyclpoly seperately seem to be ok. 
>
>However, if you have any specific tests of these three functions, 
>could you try them and see that they now give the right result,
>with the code I just checked into the CVS...
>
>Regards
>David
>  
>