test_smax_redis_client tests all now passing

src/smax/smax_client.py

@@ -6,7 +6,8 @@
 from .smax_data_types import SmaxData, \
     SmaxInt, SmaxFloat, SmaxBool, SmaxStr, SmaxStrArray, SmaxArray, SmaxStruct, \
     SmaxInt8, SmaxInt16, SmaxInt32, SmaxInt64, SmaxFloat32, SmaxFloat64, \
-    _TYPE_MAP, _REVERSE_TYPE_MAP, _SMAX_TYPE_MAP, _REVERSE_SMAX_TYPE_MAP
+    _TYPE_MAP, _REVERSE_TYPE_MAP, _SMAX_TYPE_MAP, _REVERSE_SMAX_TYPE_MAP, \
+    optional_metadata
 
 class SmaxClient(ABC):
 
@@ -150,4 +151,10 @@ def print_smax(smax_value, verbose, indent=0):
         print(indent_str, f"    date   : {datetime.datetime.utcfromtimestamp(smax_value.date)}", sep="")
         print(indent_str, f"    origin : {smax_value.origin}", sep="")
         print(indent_str, f"    seq    : {smax_value.seq}", sep="")
+        for meta in optional_metadata:
+            if hasattr(smax_value, meta):
+                if getattr(smax_value, meta) is not None:
+                    print(indent_str, f"    {meta} : {getattr(smax_value, meta)}", sep="")
+
+
 

src/smax/smax_data_types.py

@@ -58,6 +58,12 @@
 # A .data property is included for backwards compatibility with the SmaxData
 # named tuple above.
 
+optional_metadata = [
+    "description",
+    "unit",
+    "coords"
+]
+
 @dataclass
 class SmaxVarBase(object):
     """Class defining the metadata for SMA-X data types.
@@ -266,7 +272,10 @@ def __post_init__(self, *args, **kwargs):
         except AttributeError:
             pass
 
-        self.dim = self.shape
+        if len(self.shape) == 1:
+            self.dim = self.shape[0]
+        else:
+            self.dim = self.shape
         self.type = _REVERSE_TYPE_MAP[self.dtype.type]
 
     def __repr__(self):