mhostetter / galois

A performant NumPy extension for Galois fields and their applications
https://mhostetter.github.io/galois/
MIT License
321 stars 29 forks source link

Fix BCH/RS decoder caching bug #484

Closed mhostetter closed 1 year ago

mhostetter commented 1 year ago

Fixes #483

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (08e4358) 96.33% compared to head (cf9d8b4) 96.33%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/0.3.x #484 +/- ## ============================================== Coverage 96.33% 96.33% ============================================== Files 46 46 Lines 5833 5842 +9 ============================================== + Hits 5619 5628 +9 Misses 214 214 ``` | [Impacted Files](https://app.codecov.io/gh/mhostetter/galois/pull/484?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matt+Hostetter) | Coverage Δ | | |---|---|---| | [src/galois/\_codes/\_bch.py](https://app.codecov.io/gh/mhostetter/galois/pull/484?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matt+Hostetter#diff-c3JjL2dhbG9pcy9fY29kZXMvX2JjaC5weQ==) | `97.56% <100.00%> (+0.03%)` | :arrow_up: | | [src/galois/\_codes/\_reed\_solomon.py](https://app.codecov.io/gh/mhostetter/galois/pull/484?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matt+Hostetter#diff-c3JjL2dhbG9pcy9fY29kZXMvX3JlZWRfc29sb21vbi5weQ==) | `95.41% <100.00%> (+0.03%)` | :arrow_up: | | [src/galois/\_domains/\_function.py](https://app.codecov.io/gh/mhostetter/galois/pull/484?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matt+Hostetter#diff-c3JjL2dhbG9pcy9fZG9tYWlucy9fZnVuY3Rpb24ucHk=) | `97.10% <100.00%> (+0.08%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.