refactor!: remove any messagepack, leave json

README.md

@@ -69,7 +69,7 @@ poetry run pytest --export-test-cases=guppy-exports
 
 ```
 
-which will create a directory `./guppy-exports` populated with hugr modules serialised in msgpack.
+which will create a directory `./guppy-exports` populated with hugr modules serialised in JSON.
 
 ## Packaging
 

guppylang/hugr/hugr.py

@@ -794,10 +794,6 @@ def to_raw(self) -> raw.RawHugr:
             raise ValueError("Raw Hugr requires a root node")
         return raw.RawHugr(nodes=nodes, edges=edges)
 
-    def serialize(self) -> bytes:
-        """Serialize this Hugr in binary format."""
-        return self.to_raw().packb()
-
-    def serialize_json(self) -> str:
+    def serialize(self) -> str:
         """Serialize this Hugr in JSON format."""
         return self.to_raw().to_json()

guppylang/hugr/raw.py

@@ -1,6 +1,5 @@
 from typing import Any, Literal
 
-import ormsgpack
 from pydantic import BaseModel
 
 from guppylang.hugr.ops import NodeID, OpType
@@ -14,18 +13,10 @@ class RawHugr(BaseModel):
     nodes: list[OpType]
     edges: list[Edge]
 
-    def packb(self) -> bytes:
-        return ormsgpack.packb(self.model_dump(), option=ormsgpack.OPT_NON_STR_KEYS)
-
     def to_json(self) -> str:
         """Return a JSON representation of the Hugr."""
         return self.model_dump_json()
 
-    @classmethod
-    def unpackb(cls, b: bytes) -> "RawHugr":
-        """Decode a msgpack-encoded Hugr."""
-        return cls(**ormsgpack.unpackb(b, option=ormsgpack.OPT_NON_STR_KEYS))
-
     @classmethod
     def load_json(cls, json: dict[Any, Any]) -> "RawHugr":
         """Decode a JSON-encoded Hugr."""

pyproject.toml

@@ -11,7 +11,6 @@ repository = "https://github.com/CQCL/guppy"
 python = "^3.10"
 graphviz = "^0.20.1"
 networkx = "^3.2.1"
-ormsgpack = "^1.4.1"
 pydantic = "^2.5.3"
 typing-extensions = "^4.9.0"
 pytket = { version = "^1.24.0", optional = true }

tests/integration/conftest.py

@@ -23,7 +23,7 @@ def validate_json(hugr: str):
 
     def validate_impl(hugr, name=None):
         # Validate via the json encoding
-        js = hugr.serialize_json()
+        js = hugr.serialize()
         validate_json(js)
 
         if export_test_cases_dir:

validator/Cargo.toml

@@ -11,6 +11,5 @@ crate-type = ["cdylib"]
 [dependencies]
 pyo3 = "0.19.0"
 quantinuum-hugr = "0.1.0"
-rmp-serde = "1.1.1"
 lazy_static = "1.4.0"
 serde_json = "1.0.111"

validator/src/lib.rs

@@ -18,14 +18,6 @@ lazy_static! {
     .unwrap();
 }
 
-/// Validate a msgpack-encoded Hugr
-#[pyfunction]
-fn validate_bytes(hugr: Vec<u8>) -> PyResult<()> {
-    let hg: hugr::Hugr = rmp_serde::from_slice(&hugr).unwrap();
-    hg.validate(&REGISTRY).unwrap();
-    Ok(())
-}
-
 /// Validate a json-encoded Hugr
 #[pyfunction]
 fn validate_json(hugr: String) -> PyResult<()> {
@@ -36,7 +28,6 @@ fn validate_json(hugr: String) -> PyResult<()> {
 
 #[pymodule]
 fn guppyval(_py: Python, m: &PyModule) -> PyResult<()> {
-    m.add_function(wrap_pyfunction!(validate_bytes, m)?)?;
     m.add_function(wrap_pyfunction!(validate_json, m)?)?;
     Ok(())
 }