Modify display for Array.

mathworks · Sep 25, 2023 · 0d99093 · 0d99093
1 parent 979ce71
commit 0d99093
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/matlab/src/cpp/arrow/matlab/array/proxy/array.cc b/matlab/src/cpp/arrow/matlab/array/proxy/array.cc
@@ -51,8 +51,8 @@ namespace arrow::matlab::array::proxy {
         arrow::PrettyPrintOptions options;
         options.window = 3;
         options.skip_new_lines = true;
-        options.include_braces = false;
-        options.delimiter = "     ";
+        options.include_braces = true;
+        options.delimiter = ",     ";
         // MATLAB array display uses 5 spaces between elements.
         MATLAB_ERROR_IF_NOT_OK_WITH_CONTEXT(arrow::PrettyPrint(*array, options, &strs), context, error::ARRAY_DISPLAY_FAILED);
         const auto str_utf8 = strs.str();

diff --git a/matlab/src/matlab/+arrow/+array/Array.m b/matlab/src/matlab/+arrow/+array/Array.m
@@ -62,9 +62,14 @@
     end
 
     methods (Access=protected)
+
         function displayScalarObject(obj)
-            disp(obj.toString());
+            className = matlab.mixin.CustomDisplay.getClassNameForHeader(obj);
+            indent = "     ";
+            disp("Arrow " + className + " with <strong>"+ obj.Length + "</strong> elements:")
+            disp(newline + indent + obj.toString() + newline);
         end
+
     end
 
     methods