Merge pull request #40 from outofculture/issue-35-framework-integration-tests

feat: add comprehensive framework integration tests (#35)

Author: campagnola

coorx/base_transform.py

@@ -287,6 +287,8 @@ def full_matrix(self) -> np.ndarray:
 
     def as_vispy(self):
         """Return a VisPy transform that is equivalent to this transform, if possible."""
+        if self.dims != (3, 3):
+            raise NotImplementedError("as_vispy is only implemented for 3D transforms")
         from vispy.visuals.transforms import MatrixTransform
 
         # a functional default if nothing else is implemented

coorx/linear.py

@@ -418,6 +418,8 @@ def as_affine(self):
         return m
 
     def as_vispy(self):
+        if self.dims != (3, 3):
+            raise NotImplementedError("as_vispy is only implemented for 3D transforms")
         from vispy.visuals.transforms import STTransform as VispySTTransform
 
         return VispySTTransform(scale=self.scale, translate=self.offset)
@@ -854,6 +856,10 @@ def set_offset(self, offset):
         """Set the translation of this transform"""
         self.set_params(offset=offset)
 
+    @property
+    def offset(self):
+        return self._state["offset"].copy()
+
     def scale(self, scale):
         """adjust the scale of this transform"""
         ## try to prevent accidentally setting 0 scale on z axis