Repository navigation
fix: Regenerate the uint16 predictor 2 big endian fixture - #77
Merged
Merged
Conversation
uint16_1band_lzw_block128_predictor2_big_endian is rewritten from uint16_1band_lzw_block128_predictor2. #73 was branched before #76 changed that source to 256x256 with 128x128 tiles and triangle wave data, and was merged without rebasing. So the committed file still held the old 128x128 source with 64x64 tiles, and the "generated files are up to date" check on main failed. Regenerate the file and its info, and fix the docstring to give the new size and tiling. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
1 participant
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.
Note
This PR was written by Claude (Claude Code), not by @kylebarron.
CI on
mainfails at "Check generated files are up to date" (run) becauseuint16_1band_lzw_block128_predictor2_big_endianis out of date.That fixture is rewritten from
uint16_1band_lzw_block128_predictor2. #73 was branched before #76 changed that source to 256x256 with 128x128 tiles and triangle wave data, and was merged without rebasing. So the committed file was still built from the old source (128x128, 64x64 tiles).This PR:
.tifand_info.mdwithpixi run generateandpixi run info. Those are the only two files that change.The predictor 3 big endian fixture from #73 is unaffected: its source was new in #76, and it regenerates to the same bytes.
Checked locally:
pixi run checkpasses,pixi run testpasses (57 tests, includingtests/test_big_endian.py), andruff checkis clean.🤖 Written by Claude Code