Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: pure python #274

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from
Draft

WIP: pure python #274

wants to merge 14 commits into from

Conversation

joshmoore
Copy link
Member

Following on from #259 (comment), I dropped the blosc module and am slowly making changes in the background to use native blosc wheels, turning this into a pure Python module. If anyone would like to add commits, please feel free (especially if you know what you are doing unlike myself...)

TODO:

  • Unit tests and/or doctests in docstrings
  • tox -e py39 passes locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • tox -e docs passes locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Coveralls passes)

 * snappy: not compiled into wheel by default
 * decompress_partial: needs implementing
 * AUTOSHUFFLE: needs implementing
Unfortunately, this wasn't immediately put under revision control
and has undergone a number of changes. Original versions were largely
lifted out of blosc.pyx.
@pep8speaks
Copy link

pep8speaks commented Feb 18, 2021

Hello @joshmoore! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 13:34: E203 whitespace before ':'
Line 13:35: E231 missing whitespace after ':'
Line 13:39: E252 missing whitespace around parameter equals
Line 13:40: E252 missing whitespace around parameter equals

Comment last updated at 2021-02-18 15:13:38 UTC

@joshmoore joshmoore force-pushed the pure-py branch 2 times, most recently from fd05e21 to 48b277b Compare February 18, 2021 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants