Fix: AttributeArray.__str__ (#55)

* more explicit AttributeArray printing

* bump version

* add commas for array printing

* ruff format

Author: BradyAJohnston

databpy/array.py

@@ -177,8 +177,14 @@ def __repr__(self):
             obj_name = getattr(self._blender_object, "name", "Unknown")
             obj_type = getattr(self._blender_object.data, "name", "Unknown")
 
-        # Get array representation
-        array_repr = np.array_repr(np.asarray(self).view(np.ndarray))
+        # Get array representation with explicit dtype for cross-platform consistency
+        # np.array_repr() can omit dtype on Windows when it's the platform default
+        arr = np.asarray(self).view(np.ndarray)
+        # Use np.array_repr() but then ensure dtype is always appended
+        array_repr = np.array_repr(arr)
+        # If dtype isn't already in the repr, add it before the closing parenthesis
+        if f"dtype={arr.dtype}" not in array_repr:
+            array_repr = array_repr.rstrip(")") + f", dtype={arr.dtype})"
 
         return (
             f"AttributeArray(name='{attr_name}', object='{obj_name}', mesh='{obj_type}', "

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "databpy"
-version = "0.4.0"
+version = "0.4.1"
 description = "A data-oriented wrapper library for the Blender Python API"
 readme = "README.md"
 dependencies = ["numpy>=1.24.0,<2.0"]

tests/__snapshots__/test_bob.ambr

@@ -10,14 +10,14 @@
          [-1., -1., -1.]], dtype=float32)
 # ---
 # name: test_bob.1
-  array([[-1.408998  , -0.18262184,  0.88149434],
-         [-2.4226575 , -0.23832594,  0.53461844],
-         [-0.81013453,  0.6320567 , -0.12227035],
-         [ 1.1924012 , -1.5974301 , -0.14498399],
-         [ 0.7337191 , -1.6897098 ,  0.35846812],
-         [-1.0602487 , -0.74044144,  1.2087474 ],
-         [-1.6943052 ,  0.18745072, -2.2497435 ],
-         [ 1.0326254 ,  0.02491132, -0.78184235]], dtype=float32)
+  array([[-2.2497435 ,  1.0326254 ,  0.02491132],
+         [-0.78184235,  0.72800225, -0.48501468],
+         [-0.98779815, -0.15931231,  0.4114268 ],
+         [-0.7085198 , -0.2979195 ,  0.41355893],
+         [-0.6289273 , -0.9450253 ,  0.21435478],
+         [ 0.6729113 , -0.71147156, -1.161723  ],
+         [-0.03500854, -0.5114108 ,  0.31628448],
+         [-0.90172744,  0.5529607 ,  0.80140704]], dtype=float32)
 # ---
 # name: test_get_position
   array([[ 1.,  1.,  1.],
@@ -40,12 +40,12 @@
          [-1., -1., -1.]], dtype=float32)
 # ---
 # name: test_set_position.1
-  array([[-1.0290905 , -2.710574  , -0.04768289],
-         [-0.6774657 , -0.9472087 , -0.2236253 ],
-         [-1.5119076 , -1.1819721 , -0.8048682 ],
-         [-1.1509404 , -0.6400138 ,  1.2040595 ],
-         [-0.69383913, -1.5634813 ,  0.9112301 ],
-         [-0.6464026 , -1.1658763 , -0.93160886],
-         [ 0.99367285, -0.8332808 ,  0.61058164],
-         [ 0.25490105, -0.7228544 ,  0.19398543]], dtype=float32)
+  array([[ 0.61058164,  0.25490105, -0.7228544 ],
+         [ 0.19398543, -1.408998  , -0.18262184],
+         [ 0.88149434, -2.4226575 , -0.23832594],
+         [ 0.53461844, -0.81013453,  0.6320567 ],
+         [-0.12227035,  1.1924012 , -1.5974301 ],
+         [-0.14498399,  0.7337191 , -1.6897098 ],
+         [ 0.35846812, -1.0602487 , -0.74044144],
+         [ 1.2087474 , -1.6943052 ,  0.18745072]], dtype=float32)
 # ---

tests/test_array_print_methods.py

@@ -115,6 +115,39 @@ def test_str_method_with_large_array(self):
         # Should contain numpy's truncation indicator for large arrays
         assert "..." in result or len(result.split("\n")) > 1
 
+    def test_repr_always_shows_dtype(self, blender_object):
+        """Test that __repr__ always explicitly shows dtype for cross-platform consistency.
+
+        This is important for snapshot testing across platforms. On Windows, np.array_repr()
+        may omit dtype when it's the platform default (e.g., int32), while on macOS/Linux
+        it's always shown. We ensure dtype is always explicit in our repr output.
+        """
+        from databpy.attribute import store_named_attribute
+
+        # Get the number of points on the blender object
+        num_points = len(blender_object.data.vertices)
+
+        # Test with int32 (the type that causes platform-dependent repr on Windows)
+        int_data = np.ones(num_points, dtype=np.int32)
+        store_named_attribute(blender_object, int_data, "test_int32")
+        int_arr = AttributeArray(blender_object, "test_int32")
+
+        int_repr = repr(int_arr)
+        # Should contain 'dtype=int32' or 'dtype=int32)' somewhere in output
+        assert "dtype=int32" in int_repr, (
+            f"int32 dtype not explicitly shown in repr: {int_repr}"
+        )
+
+        # Test with float32 as well
+        float_data = np.ones(num_points, dtype=np.float32)
+        store_named_attribute(blender_object, float_data, "test_float32")
+        float_arr = AttributeArray(blender_object, "test_float32")
+
+        float_repr = repr(float_arr)
+        assert "dtype=float32" in float_repr, (
+            f"float32 dtype not explicitly shown in repr: {float_repr}"
+        )
+
 
 class TestColumnSlicePrintMethods:
     """Test print behavior of column slice views."""