fix(io.ec_data): satisfy ruff (UP037, B905, I001)

- UP037: drop the string-quoted forward reference on EcData.empty's
  return annotation; the module already uses `from __future__ import
  annotations`, so the bare class name is fine.
- B905: zip(coo.row, coo.col, coo.data) now passes strict=True. The
  three arrays come from the same COO matrix and are guaranteed
  equal length; strict=True turns any future drift into a loud
  TypeError instead of silent truncation.
- I001: drop the stray blank line between the import block and the
  first section comment in tests/test_io_yaml_ec_data.py.

Author: edkerk

src/raven_python/io/ec_data.py

@@ -153,7 +153,7 @@ def validate(self) -> None:
 
     @staticmethod
     def empty(n_rxns: int, n_enzymes: int = 0, *,
-              gecko_light: bool = False) -> "EcData":
+              gecko_light: bool = False) -> EcData:
         """Preallocate an ``EcData`` with the canonical sentinel values.
 
         Per-rxn fields get empty strings; per-enzyme fields get empty
@@ -311,7 +311,10 @@ def _build_ec_rxns_list(ec: EcData) -> list[dict[str, Any]]:
     """
     coo = ec.rxn_enz_mat.tocoo()
     per_row_enzymes: list[dict[str, float]] = [{} for _ in range(ec.n_rxns)]
-    for i, j, v in zip(coo.row, coo.col, coo.data):
+    # All three arrays come from the same COO matrix, so they're guaranteed
+    # equal length; strict=True turns any future drift into a loud TypeError
+    # instead of silent truncation.
+    for i, j, v in zip(coo.row, coo.col, coo.data, strict=True):
         per_row_enzymes[int(i)][ec.enzymes[int(j)]] = float(v)
 
     out: list[dict[str, Any]] = []

tests/test_io_yaml_ec_data.py

@@ -23,7 +23,6 @@
 from raven_python.io import EcData, read_yaml_model, write_yaml_model
 from raven_python.io.yaml import model_from_yaml_data
 
-
 # --------------------------------------------------------------------------- #
 # Helpers
 # --------------------------------------------------------------------------- #