Merge pull request #1404 from NiklasGustafsson/bugs

Fixed npstr() of empty tensor()

Author: NiklasGustafsson

src/TorchSharp/Tensor/Tensor.cs

@@ -6506,16 +6506,18 @@ private static string ToNumpyString(Tensor t, long mdim, bool isFCreate, string
 
                 var dim = t.dim();
 
-                if (t.size().Length == 0) return "";
                 var sb = new StringBuilder(isFCreate ? string.Join("", Enumerable.Repeat(' ', (int)(mdim - dim))) : "");
+
+                if (dim == 0) {
+                    PrintValue(sb, t.dtype, t.ToScalar(), fltFormat, actualCulturInfo);
+                    return sb.ToString(); ;
+                }
+
                 sb.Append('[');
                 var currentSize = t.size()[0];
                 if (currentSize == 0) {
                     // print nothing
                 }
-                else if (dim == 0) {
-                    PrintValue(sb, t.dtype, t.ToScalar(), fltFormat, actualCulturInfo);
-                }
                 else if (dim == 1) {
                     if (currentSize <= torch.maxColumns) {
                         for (var i = 0; i < currentSize - 1; i++) {

test/TorchSharpTest/TestTorchTensorBugs.cs

@@ -1660,6 +1660,15 @@ public void Validate_1250()
             Assert.Equal("[1], type = Float32, device = cpu, value = float [] {0f}", torch.zeros(1).cstr());
         }
 
+        [Fact]
+        public void Validate_1250a()
+        {
+            var scalar = torch.zeros(Array.Empty<long>());
+            Assert.Equal("0", scalar.npstr());
+            Assert.Equal("[], type = Float32, device = cpu, value = 0", scalar.cstr());
+            Assert.Equal("[], type = Float32, device = cpu, value = 0", scalar.jlstr());
+        }
+
         [Fact]
         public void ValidateLoadWithDeflateStream()
         {