search

tomerfiliba-org / reedsolomon

⏳🛡 Pythonic universal errors-and-erasures Reed-Solomon codec to protect your data from errors and bitrot. Includes a future-proof zero-dependencies pure-python implementation 🔮 and an optional speed-optimized Cython/C extension 🚀
http://pypi.python.org/pypi/reedsolo
Other
351 stars 86 forks source link

Replace [::-1] step by explicit inversion function #67

Closed elvis-epx closed 12 months ago

elvis-epx commented 12 months ago

This change makes the module compatible with MicroPython, which does not support stepped slices. Tested with tests/test_reedsolo.py.

lrq3000 commented 12 months ago

Thank you very much for your PR! Please just add the docstring so that future maintainers keep in mind why this is necessary and hence qvoid regressions, and then I'll merge and release a new beta soon on pypi :-)

coveralls commented 12 months ago

Pull Request Test Coverage Report for Build 5538533195

Totals Coverage Status
Change from base Build 4952845024: 0.03%
Covered Lines: 430
Relevant Lines: 447
💛 - Coveralls
elvis-epx commented 12 months ago

Thank you very much for your PR! Please just add the docstring so that future maintainers keep in mind why this is necessary and hence qvoid regressions, and then I'll merge and release a new beta soon on pypi :-)

Done!

coveralls commented 12 months ago

Pull Request Test Coverage Report for Build 5544999920

Totals Coverage Status
Change from base Build 4952845024: 0.02%
Covered Lines: 429
Relevant Lines: 446
💛 - Coveralls
coveralls commented 12 months ago

Pull Request Test Coverage Report for Build 5544999920

Totals Coverage Status
Change from base Build 4952845024: 0.03%
Covered Lines: 430
Relevant Lines: 447
💛 - Coveralls
coveralls commented 12 months ago

Pull Request Test Coverage Report for Build 5544999920

Totals Coverage Status
Change from base Build 4952845024: 0.3%
Covered Lines: 431
Relevant Lines: 447
💛 - Coveralls
lrq3000 commented 12 months ago

Perfect, thank you so much! 😃

coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 5544999920

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

Totals Coverage Status
Change from base Build 4952845024: 0.3%
Covered Lines: 431
Relevant Lines: 447
💛 - Coveralls
coveralls commented 3 months ago

Pull Request Test Coverage Report for Build 5544999920

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

Totals Coverage Status
Change from base Build 4952845024: 0.3%
Covered Lines: 431
Relevant Lines: 447
💛 - Coveralls