Skip to content

Commit 8f99d4a

Browse files
committed
Soften incorrect packbits-only-uint8 claim in test commentary
Copilot flagged the docstring and section comment claiming packbits in this writer only supports uint8. packbits operates on the raw byte stream, so any dtype is technically supported; uint8 is just the variant this test exercises (and the dtype packbits was designed for).
1 parent e88175d commit 8f99d4a

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

xrspatial/geotiff/tests/test_streaming_codecs_2026_05_11.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,10 @@
99
* Dask streaming write + LERC (lossless and lossy ``max_z_error``)
1010
produces identical output to the eager writer.
1111
* Dask streaming write + LZ4 round-trips a float32 raster.
12-
* Dask streaming write + packbits round-trips a uint8 raster (the only
13-
dtype packbits supports in this writer).
12+
* Dask streaming write + packbits round-trips a uint8 raster. packbits
13+
operates on the raw byte stream so any dtype is technically supported;
14+
uint8 is the variant exercised here and the one packbits was designed
15+
for (run-length encoding of byte runs).
1416
* COG output with ``overview_resampling='cubic'`` round-trips through
1517
scipy.ndimage.zoom (the only overview method that takes a separate
1618
code path in ``_block_reduce_2d``).
@@ -139,7 +141,7 @@ def test_streaming_matches_eager(self, float_raster, dask_float_raster,
139141

140142

141143
# ---------------------------------------------------------------------------
142-
# packbits: round-trip on uint8 (the only supported dtype)
144+
# packbits: round-trip on uint8 (the dtype packbits is designed for)
143145
# ---------------------------------------------------------------------------
144146

145147
class TestStreamingPackbits:

0 commit comments

Comments
 (0)