Update dependency Blosc/c-blosc2 to v3.2.3 - #73
Open
wongcht wants to merge 1 commit into
Open
Conversation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR contains the following updates:
3.2.1→3.2.3Release Notes
Blosc/c-blosc2 (Blosc/c-blosc2)
v3.2.3Compare Source
===========================
#XXX version-specific blurb XXX#
Bug fixes
Fixed wrong results in schunk slice and sparse reads for typesize > 255.
blosc2_schunk_get_slice_buffer()derived itsblosc2_getitem_ctx()item counts by dividing byte offsets by
schunk->typesize, but chunkswhose typesize exceeds
BLOSC_MAX_TYPESIZE(255) are compressed with aninternal typesize of 1, so getitem counts bytes for them. Slices that did
not cover a chunk exactly mostly failed with
BLOSC2_ERROR_FAILURE, andsingle-element slices returned the wrong bytes with a success return code.
The single-coordinate path of
blosc2_schunk_get_sparse_buffer()had thesame confusion. Both now convert through the chunk's actual item unit.
See #796.
blosc2_getitem_ctx()now returnsBLOSC2_ERROR_DATAinstead of ashort byte count when it cannot decode every requested item, so a partial
decode no longer looks like a success to callers that only test for a
negative return.
v3.2.2Compare Source
===========================
Bug fixes
compact get path introduced in 3.2.2 passed item counts to
blosc2_getitem_ctx()in array-typesize units, but chunks whosetypesize exceeds
BLOSC_MAX_TYPESIZE(255) are compressed with aninternal typesize of 1, so small slice reads returned truncated blocks
(only the first
blocknitemsbytes were correct). Item counts arenow expressed in the chunk's actual item unit, keeping the fast path
both correct and O(request) for large typesizes. Added a regression
test covering typesize > 255 on the get_slice path.
Notes
reading slices from b2nd arrays with typesize > 255 should upgrade.
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate.