small fixes

Author: tomvanmele

CHANGELOG.md

@@ -15,12 +15,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Added optional `path` parameter to `compas.rpc.Proxy` to allow for non-package calls.
 * Added Grasshopper component to call RPC functions.
 * Added `Mesh.to_lines` method and tests.
-* Added `Data.guid` to serialization.
+* Added `Data.guid` to JSON serialization.
+* Added `Data.guid` to pickle state.
 
 ### Changed
 
 * Set `jinja >= 3.0` to dev dependencies to fix docs build error.
 * Fixed removing of collections for `compas_plotters`.
+* Fixed bug in `compas.robots.Configuration`.
 
 ### Removed
 

src/compas/data/data.py

@@ -6,6 +6,7 @@
 import json
 import hashlib
 from uuid import uuid4
+from uuid import UUID
 from copy import deepcopy
 
 import compas
@@ -113,12 +114,19 @@ def __init__(self, name=None):
 
     def __getstate__(self):
         """Return the object data for state serialization with older pickle protocols."""
-        return {"__dict__": self.__dict__, "dtype": self.dtype, "data": self.data}
+        return {
+            "__dict__": self.__dict__,
+            "dtype": self.dtype,
+            "data": self.data,
+            "guid": str(self.guid),
+        }
 
     def __setstate__(self, state):
         """Assign a deserialized state to the object data to support older pickle protocols."""
         self.__dict__.update(state["__dict__"])
         self.data = state["data"]
+        if "guid" in state:
+            self._guid = UUID(state['guid'])
 
     @property
     def DATASCHEMA(self):

src/compas/robots/configuration.py

@@ -134,7 +134,9 @@ class Configuration(Data):
 
     """
 
-    def __init__(self, joint_values=None, joint_types=None, joint_names=None):
+    def __init__(self, joint_values=None, joint_types=None, joint_names=None, **kwargs):
+        super(Configuration, self).__init__(**kwargs)
+
         joint_values = FixedLengthList(joint_values or [])
         joint_types = FixedLengthList(joint_types or [])
         joint_names = FixedLengthList(joint_names or [], validator=joint_names_validator)

tests/compas/data/test_pickle.py

@@ -10,3 +10,4 @@ def test_pickling():
     assert all(a == b for a, b in zip(f1.xaxis, f2.xaxis))
     assert all(a == b for a, b in zip(f1.yaxis, f2.yaxis))
     assert all(a == b for a, b in zip(f1.zaxis, f2.zaxis))
+    assert f1.guid == f2.guid