-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
gh-132983: Add compression.zstd
and Python tests
#133365
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
f4cd026
Add Python files
emmatyping 4168895
Fix byteswarning in test
emmatyping a22fa9b
Remove shape tests
emmatyping e70e03b
Make namedtuples dataclasses
emmatyping cbf0ef8
Apply suggestions from AA-Turner
emmatyping 298b369
Clean up chunk calculations in train_dict
emmatyping a22be68
Fix _CLValues instantiation
emmatyping b30ed02
More cleanup of train_/finalize_dict
emmatyping 307a894
Have train_/finalize_dict take tuple not list
emmatyping dd716a4
Ensure trailing data raises errors
emmatyping 9b4765b
Remove paramter bounds caching and unsupported...
emmatyping 214cd60
Use kwargs for code clarity
emmatyping e1f53b1
Clean up imports in zstd tests
emmatyping 4c00026
Use _1K instead of 1024 in tests
emmatyping 99653d2
Move compression.zstd.zstdfile to compression.zstd._zstdfile
emmatyping e403a25
Change compressLevel_values to COMPRESSION_LEVEL_DEFAULT
emmatyping 63625bc
Fix tests for change in error message
emmatyping 1ea4b9a
Make parameter names snake case
emmatyping ad05da8
Replace compressionLevel_values re-export with COMPRESSION_LEVEL_DEFAULT
emmatyping e82e23d
Move zstd_support_multithread to tests and rename
emmatyping 7801b6b
Update module docstring for compression.zstd
emmatyping df5d827
Clarify Strategy stability in docstring
emmatyping 4ff48da
Fix formatting in tarfile
emmatyping c68a896
Remove zstd_support_multithread from __all__
emmatyping 326400d
Add test_name from upstream
emmatyping 2c0c9a1
Don't close tarfile if there is a BaseException
emmatyping 49f3821
Use options kwarg in tests
emmatyping 8ba6bda
Use options kwarg in tests in more places
emmatyping 129d5e6
Adopt suggestions by Tomas R. for _zstdfile
emmatyping 7d54d35
Formatting fixes in zstd tests
emmatyping 03795ec
Improve docstrings for (de)compress
emmatyping 01fcfcb
Fix some line length issues
emmatyping f04494c
Improve docstring on C/DParameter.bounds()
emmatyping caa40b1
Improve docstrings and formatting
emmatyping 4584ec5
Add missing f string prefix
emmatyping 3cafdc6
Fix weird indent in _zstdfile.py
emmatyping 8cb0846
Use io.open instead of builtins.open
emmatyping c7d5d67
Remove _READER_CLASS from ZstdFile
emmatyping a56a22e
Adopt many suggestions from AA-Turner for ZstdFile
emmatyping 7e919c8
Set self._buffer to None
emmatyping 389faed
Move _nbytes to _zstdfile.py
emmatyping 006ef2e
Move test_zstd to file
emmatyping c846b78
Rename C/DParameter to (De)CompressionParameter
emmatyping fa0cb0c
regen clinic
AA-Turner 74e4d2b
Fix whitespace issue
emmatyping 03fff3d
Remove makefile test dir
emmatyping a99c5dd
swap order of parameters in _get_param_bounds
AA-Turner a12a031
Merge branch 'main' into 3.14-zstd-python-code
emmatyping bf94aad
Sort imports
AA-Turner 5b45ec7
Improve docstrings
AA-Turner b0eca5a
Remove comments
AA-Turner c0d0e10
Remove unused private variables
AA-Turner 10f0cff
Misc changes (positional-only, style, error messages)
AA-Turner 7f8c350
whitespace
AA-Turner bf4b07d
Remove _set_parameter_types
AA-Turner eaf46a8
Revert "Remove _set_parameter_types"
emmatyping File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Improve docstrings and formatting
- Loading branch information
commit caa40b192ca63dbcfa98ea4ad5ed9ed00b77677d
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.