splitgrid 0.1.1__tar.gz → 0.1.2__tar.gz

{splitgrid-0.1.1 → splitgrid-0.1.2}/Makefile

@@ -1,6 +1,6 @@
 # SplitGrid
 
-.PHONY: test test-all test-slow verify install-test
+.PHONY: test test-all test-slow verify install-test bench bench-child profile-pack
 
 # Prefer a local venv so `make test` / `make verify` work without activating it.
 PYTEST := $(if $(wildcard .venv/bin/pytest),.venv/bin/pytest,python3 -m pytest)
@@ -22,3 +22,16 @@ test-slow:
 # deal contracts + hypothesis round-trips (excludes CrossHair subprocess).
 verify:
 	$(PYTEST) tests/test_serialization_verification.py tests/test_payload_codec_policy_verification.py -m "not slow" -q
+
+PYTHON := $(if $(wildcard .venv/bin/python),.venv/bin/python,python3)
+
+# Asymmetric host/child serialization bench (needs numpy).
+bench:
+	$(PYTHON) scripts/bench_serialization.py --direction both
+
+bench-child:
+	$(PYTHON) scripts/bench_serialization.py --child-only
+
+profile-pack:
+	$(PYTHON) scripts/profile_pack.py
+	$(PYTHON) scripts/profile_nones.py

{splitgrid-0.1.1/src/splitgrid.egg-info → splitgrid-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: splitgrid
-Version: 0.1.1
+Version: 0.1.2
 Summary: Asymmetric split-grid serialization: host stdlib flatten + optional Cython, child NumPy unpack
 Author-email: Keith Cu <keithcu@gmail.com>
 License: GPL-3.0-or-later
@@ -35,6 +35,8 @@ Dynamic: license-file
 
 # SplitGrid
 
+[![PyPI](https://img.shields.io/pypi/v/splitgrid.svg)](https://pypi.org/project/splitgrid/)
+
 Asymmetric **split-grid** serialization for rectangular numeric and mixed-type grids.
 
 This package was **pulled out of [WriterAgent](https://github.com/KeithCu/writeragent)** (LibreOffice Writer/Calc/Draw AI extension). It is the host flatten + child unpack codec that lived in `plugin.scripting.payload_codec`, plus the optional Cython flatten accelerator from `native/writeragent_vec`.
@@ -137,6 +139,20 @@ make verify             # deal contracts + hypothesis round-trips, no CrossHair
 
 Default pytest **does not require** the compiled extension. When `splitgrid.pack` is built, `tests/test_cython_parity.py` compares Cython and Python flatten components on the same grids.
 
+## Bench
+
+Requires NumPy (and the optional Cython extension if you want the accelerated pack row). From a source checkout:
+
+```bash
+python scripts/bench_serialization.py --direction both
+python scripts/bench_serialization.py --child-only
+python scripts/bench_unpacking_opt.py
+python scripts/profile_pack.py
+python scripts/profile_nones.py
+python scripts/run_serialization_ab.py --all
+make bench
+```
+
 ## Public API
 
 ```python

{splitgrid-0.1.1 → splitgrid-0.1.2}/README.md

@@ -1,5 +1,7 @@
 # SplitGrid
 
+[![PyPI](https://img.shields.io/pypi/v/splitgrid.svg)](https://pypi.org/project/splitgrid/)
+
 Asymmetric **split-grid** serialization for rectangular numeric and mixed-type grids.
 
 This package was **pulled out of [WriterAgent](https://github.com/KeithCu/writeragent)** (LibreOffice Writer/Calc/Draw AI extension). It is the host flatten + child unpack codec that lived in `plugin.scripting.payload_codec`, plus the optional Cython flatten accelerator from `native/writeragent_vec`.
@@ -102,6 +104,20 @@ make verify             # deal contracts + hypothesis round-trips, no CrossHair
 
 Default pytest **does not require** the compiled extension. When `splitgrid.pack` is built, `tests/test_cython_parity.py` compares Cython and Python flatten components on the same grids.
 
+## Bench
+
+Requires NumPy (and the optional Cython extension if you want the accelerated pack row). From a source checkout:
+
+```bash
+python scripts/bench_serialization.py --direction both
+python scripts/bench_serialization.py --child-only
+python scripts/bench_unpacking_opt.py
+python scripts/profile_pack.py
+python scripts/profile_nones.py
+python scripts/run_serialization_ab.py --all
+make bench
+```
+
 ## Public API
 
 ```python

{splitgrid-0.1.1 → splitgrid-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "splitgrid"
-version = "0.1.1"
+version = "0.1.2"
 description = "Asymmetric split-grid serialization: host stdlib flatten + optional Cython, child NumPy unpack"
 readme = "README.md"
 requires-python = ">=3.11"

{splitgrid-0.1.1 → splitgrid-0.1.2}/src/splitgrid/__init__.py

@@ -55,7 +55,7 @@ from splitgrid.codec import (
     wire_cell_count,
 )
 
-__version__ = "0.1.1"
+__version__ = "0.1.2"
 
 
 def __getattr__(name: str):

{splitgrid-0.1.1 → splitgrid-0.1.2}/src/splitgrid/codec.py

@@ -813,6 +813,10 @@ def _flatten_update_column_state(column_states: list[int], c: int, val: Any) ->
             column_states[c] = 2
     elif tname.startswith("float"):
         column_states[c] = 3
+    else:
+        # Decimal/Fraction/etc. already survived float(val). Default to float,
+        # matching Cython _update_column_state — not int (state 0).
+        column_states[c] = 3
 
 
 def _flatten_append_cell_slow(
@@ -1421,7 +1425,16 @@ def _child_unpack_single_data(wire: Any) -> Any:
         else:
             grid = list(unpacked)
         if is_numeric_grid(grid):
-            arr = np.array(grid, dtype=np.float64)
+            # is_numeric_coercible treats whitespace/"" as Calc blanks, but
+            # np.float64 cannot convert those strings (ValueError). Keep the list.
+            try:
+                arr = np.array(grid, dtype=np.float64)
+            except ValueError:
+                log.debug(
+                    "splitgrid child_unpack json_list as-is (non-floatable blanks) %s",
+                    describe_wire_value(unpacked),
+                )
+                return grid
             log.debug(
                 "splitgrid child_unpack json_list -> ndarray shape=%s",
                 arr.shape,

{splitgrid-0.1.1 → splitgrid-0.1.2/src/splitgrid.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: splitgrid
-Version: 0.1.1
+Version: 0.1.2
 Summary: Asymmetric split-grid serialization: host stdlib flatten + optional Cython, child NumPy unpack
 Author-email: Keith Cu <keithcu@gmail.com>
 License: GPL-3.0-or-later
@@ -35,6 +35,8 @@ Dynamic: license-file
 
 # SplitGrid
 
+[![PyPI](https://img.shields.io/pypi/v/splitgrid.svg)](https://pypi.org/project/splitgrid/)
+
 Asymmetric **split-grid** serialization for rectangular numeric and mixed-type grids.
 
 This package was **pulled out of [WriterAgent](https://github.com/KeithCu/writeragent)** (LibreOffice Writer/Calc/Draw AI extension). It is the host flatten + child unpack codec that lived in `plugin.scripting.payload_codec`, plus the optional Cython flatten accelerator from `native/writeragent_vec`.
@@ -137,6 +139,20 @@ make verify             # deal contracts + hypothesis round-trips, no CrossHair
 
 Default pytest **does not require** the compiled extension. When `splitgrid.pack` is built, `tests/test_cython_parity.py` compares Cython and Python flatten components on the same grids.
 
+## Bench
+
+Requires NumPy (and the optional Cython extension if you want the accelerated pack row). From a source checkout:
+
+```bash
+python scripts/bench_serialization.py --direction both
+python scripts/bench_serialization.py --child-only
+python scripts/bench_unpacking_opt.py
+python scripts/profile_pack.py
+python scripts/profile_nones.py
+python scripts/run_serialization_ab.py --all
+make bench
+```
+
 ## Public API
 
 ```python

{splitgrid-0.1.1 → splitgrid-0.1.2}/tests/test_payload_codec.py

@@ -15,6 +15,7 @@ from __future__ import annotations
 
 import ast
 import math
+from decimal import Decimal
 from pathlib import Path
 from unittest.mock import patch
 
@@ -41,6 +42,7 @@ from splitgrid.codec import (
     should_use_binary_envelope,
     wire_cell_count,
 )
+from tests.serialization_ab_support import cython_accelerator_context
 from tests.payload_codec_test_support import (
     MIXED_LABEL_GRID,
     MIXED_WITH_ZIP,
@@ -512,6 +514,43 @@ def test_mixed_grid_preserves_non_numeric_string() -> None:
     assert out[0][1] == "hello"
 
 
+def test_whitespace_only_cell_does_not_crash_child_unpack() -> None:
+    """Pasted '   ' is numeric-coercible for Calc but np.float64 cannot convert it."""
+    pytest.importorskip("numpy")
+    grid = [[1.0, "   ", 3.0, 4.0], [5.0, 6.0, 7.0, 8.0], [9.0, 10.0, 11.0, 12.0]]
+    out = child_unpack_data(host_pack_data(grid, force="always"))
+    assert isinstance(out, list)
+    assert out[0][1] == "   "
+
+
+def test_empty_string_mixed_split_grid_does_not_crash_child_unpack() -> None:
+    """Bare '' on mixed split_grid must not raise (Calc usually maps '' to None first)."""
+    pytest.importorskip("numpy")
+    grid = [[1.0, "", 3.0, 4.0], [5.0, 6.0, 7.0, 8.0], [9.0, 10.0, 11.0, 12.0]]
+    out = child_unpack_data(host_pack_data(grid, force="always"))
+    assert isinstance(out, list)
+    assert out[0][1] == ""
+
+
+def test_mixed_grid_real_nan_becomes_none_on_child() -> None:
+    """Documented: mixed-grid ingress has no blank-vs-NaN wire bit."""
+    pytest.importorskip("numpy")
+    grid = [[1.0, float("nan")], ["label", 4.0]]
+    out = child_unpack_data(host_pack_data(grid, force="always"))
+    assert out[0][1] is None
+    assert out[1][0] == "label"
+
+
+def test_decimal_split_grid_stays_float_not_truncated_int() -> None:
+    """stdlib flatten must label Decimal columns float (Cython already did)."""
+    pytest.importorskip("numpy")
+    grid = [[Decimal("1.5"), Decimal("2.25")], [Decimal("3.0"), Decimal("4.75")]]
+    with cython_accelerator_context(enabled=False):
+        out = child_unpack_data(host_pack_data(grid, force="always"))
+    assert out[0][0] == pytest.approx(1.5)
+    assert out[0][1] == pytest.approx(2.25)
+
+
 def test_bool_cells_round_trip_in_numeric_grid() -> None:
     """Calc booleans in an all-numeric grid become 0.0/1.0 in child ndarray (float64 lane)."""
     np = pytest.importorskip("numpy")