Add deprication warnings to non-float Coordinates

Author: pattonw

src/funlib/geometry/__init__.py

@@ -1,10 +1,26 @@
-from .coordinate import Coordinate  # noqa
+import warnings
+
+from .coordinate import Coordinate as _Coordinate  # noqa
 from .float_coordinate import FloatCoordinate  # noqa
 from .float_roi import FloatRoi  # noqa
 from .roi import Roi  # noqa
 
+__all__ = ["Coordinate", "FloatCoordinate", "Roi", "FloatRoi"]
+
 __major__ = 0
 __minor__ = 3
 __patch__ = 0
 __tag__ = ""
 __version__ = "{}.{}.{}{}".format(__major__, __minor__, __patch__, __tag__).strip(".")
+
+
+def __getattr__(name):
+    if name == "Coordinate":
+        warnings.warn(
+            "Coordinate is deprecated, use FloatCoordinate instead. "
+            "Coordinate will be removed in a future version.",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        return _Coordinate
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

src/funlib/geometry/coordinate.py

@@ -137,6 +137,11 @@ def __mul__(self, other: Union[Any, "Coordinate", int, float]) -> "Coordinate":
 
     def __div__(self, other: Union["Coordinate", int, float]) -> "Coordinate":
         if isinstance(other, Coordinate):
+            if type(self) is not type(other):
+                raise TypeError(
+                    f"cannot use / between {type(self).__name__} and "
+                    f"{type(other).__name__}; convert to the same type or use //"
+                )
             assert self.dims == other.dims, (
                 "can only divide Coordinate of equal dimensions"
             )
@@ -156,6 +161,11 @@ def __div__(self, other: Union["Coordinate", int, float]) -> "Coordinate":
 
     def __truediv__(self, other: Union["Coordinate", int, float]) -> "Coordinate":
         if isinstance(other, Coordinate):
+            if type(self) is not type(other):
+                raise TypeError(
+                    f"cannot use / between {type(self).__name__} and "
+                    f"{type(other).__name__}; convert to the same type or use //"
+                )
             assert self.dims == other.dims, (
                 "can only divide Coordinate of equal dimensions"
             )