ipodsync 0.1.3__tar.gz → 0.1.4__tar.gz

{ipodsync-0.1.3 → ipodsync-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ipodsync
-Version: 0.1.3
+Version: 0.1.4
 Summary: Upload music to an iPod nano 6G/7G without iTunes: SQLite library, hashAB signing, cover art (pure-Python).
 Project-URL: Homepage, https://github.com/buldiei/ipodsync
 Project-URL: Repository, https://github.com/buldiei/ipodsync

{ipodsync-0.1.3 → ipodsync-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "ipodsync"
-version = "0.1.3"
+version = "0.1.4"
 description = "Upload music to an iPod nano 6G/7G without iTunes: SQLite library, hashAB signing, cover art (pure-Python)."
 readme = "README.md"
 requires-python = ">=3.9"

{ipodsync-0.1.3 → ipodsync-0.1.4}/src/ipodsync/__init__.py

@@ -13,7 +13,7 @@ from .library import ItlpLibrary
 from .models import Track
 from .transport import IPod, IPodNotFound, find_ipod
 
-__version__ = "0.1.3"
+__version__ = "0.1.4"
 
 __all__ = [
     "find_ipod",

{ipodsync-0.1.3 → ipodsync-0.1.4}/src/ipodsync/artwork_writer.py

@@ -58,20 +58,30 @@ FIRST_IMAGE_ID = 101
 
 # --- encoding a cover to RGB565-LE ----------------------------------------------
 def to_rgb565(image_bytes: bytes, width: int, height: int, slot: int) -> bytes:
-    """Resize the cover to width×height, pack into RGB565-LE, pad to slot."""
+    """Resize the cover to width×height, pack into RGB565-LE.
+
+    Each row is padded to the device's row stride (`slot // height`), matching
+    iTunes: e.g. F1016 is 57×57 pixels but stored with a 116-byte (58 px) stride —
+    2 zero bytes at the end of every row. Getting this wrong shears the thumbnail.
+    """
     from PIL import Image
 
     im = Image.open(io.BytesIO(image_bytes)).convert("RGB").resize(
         (width, height), Image.LANCZOS)
     rgb = im.tobytes()  # width*height*3
-    out = bytearray(width * height * 2)
-    j = 0
-    for i in range(0, len(rgb), 3):
-        v = ((rgb[i] >> 3) << 11) | ((rgb[i + 1] >> 2) << 5) | (rgb[i + 2] >> 3)
-        out[j] = v & 0xFF
-        out[j + 1] = v >> 8
-        j += 2
-    if len(out) < slot:
+    stride = slot // height          # bytes per row the device reads
+    out = bytearray()
+    for y in range(height):
+        row = bytearray(stride)      # zero-padded to the stride
+        base, j = y * width * 3, 0
+        for _ in range(width):
+            v = ((rgb[base] >> 3) << 11) | ((rgb[base + 1] >> 2) << 5) | (rgb[base + 2] >> 3)
+            row[j] = v & 0xFF
+            row[j + 1] = v >> 8
+            base += 3
+            j += 2
+        out += row
+    if len(out) < slot:              # safety (slot == height*stride already)
         out.extend(b"\x00" * (slot - len(out)))
     return bytes(out)
 

{ipodsync-0.1.3 → ipodsync-0.1.4}/tests/test_artwork_writer.py

@@ -137,3 +137,17 @@ def test_row_or_template_fallback_on_empty():
     got["title"] = "x"
     assert library._EMPTY_TEMPLATES["item"].get("title") != "x"
     c.close()
+
+
+def test_f1016_row_stride_padding():
+    """F1016 is 57×57 pixels but stored with a 116-byte (58 px) row stride — two
+    zero bytes at the end of every row, like iTunes. Wrong stride shears the thumb."""
+    from PIL import Image
+    import io
+    buf = io.BytesIO()
+    Image.new("RGB", (80, 80), (200, 30, 30)).save(buf, format="PNG")
+    data = aw.to_rgb565(buf.getvalue(), 57, 57, 6612)
+    assert len(data) == 6612
+    stride = 6612 // 57  # 116
+    for y in range(57):
+        assert data[y * stride + 114:y * stride + 116] == b"\x00\x00", f"row {y}"