Make fcr = 128 - T for CCSDS, i.e. 112 for 255,223 and 120 for 255,239

pjkundert / ezpwd-reed-solomon

Reed-Solomon & BCH encoding and decoding, in C++, Javascript & Python

https://hardconsulting.com/products/13-reed-solomon

Other

99 stars 21 forks source link

Make fcr = 128 - T for CCSDS, i.e. 112 for 255,223 and 120 for 255,239 #6

Closed jlowder closed 7 years ago

jlowder commented 7 years ago

CCSDS allows both 255,223 and 255,239: the fcr needs to be 112 for 255,223 but 120 for the 255,239 case.

pjkundert commented 7 years ago

Thanks for this! I've begun a general modification to more correctly support the limited number of valid CCSDS encodings, and their associated Berleskamp "dual basis" representation. However, it is a work in progress, not yet complete... See branch feature-CCSDS

jlowder commented 7 years ago

That sounds great - I'm glad to hear you're working on adding dual-basis support. So far I have only come across proprietary implementations of that, so it will be great to have an open-source alternative.