Simpler: pass to numpy in all cases

Author: fmaussion

xarray/core/common.py

@@ -156,10 +156,7 @@ def _repr_html_(self):
         return formatting_html.array_repr(self)
 
     def __format__(self: Any, format_spec: str) -> str:
-        if not format_spec:
-            # format without specifier falls backs to standard repr
-            return formatting.array_repr(self)
-        # else use numpy: scalars will print fine and arrays will raise
+        # we use numpy: scalars will print fine and arrays will raise
         return self.values.__format__(format_spec)
 
     def _iter(self: Any) -> Iterator[Any]:

xarray/tests/test_formatting.py

@@ -415,15 +415,12 @@ def test_array_repr_variable(self) -> None:
 
     def test_array_scalar_format(self) -> None:
         var = xr.DataArray(0)
-        assert var.__format__("") == "<xarray.DataArray ()>\narray(0)"
+        assert var.__format__("") == "0"
         assert var.__format__("d") == "0"
         assert var.__format__(".2f") == "0.00"
 
-        var = xr.DataArray([0])
-        assert var.__format__("") == (
-            "<xarray.DataArray (dim_0: 1)>\narray([0])"
-            "\nDimensions without coordinates: dim_0"
-        )
+        var = xr.DataArray([0.1, 0.2])
+        assert var.__format__("") == "[0.1 0.2]"
         with pytest.raises(TypeError) as excinfo:
             var.__format__(".2f")
         assert "unsupported format string passed to" in str(excinfo.value)