Merge pull request #35 from cedadev/fix34

Fix34

Author: dwest77a

cfapyx/backend.py

@@ -21,7 +21,7 @@
 def open_cfa_dataset(
         filename_or_obj,
         drop_variables=None,
-        mask_and_scale=None,
+        mask_and_scale=True,
         decode_times=True,
         concat_characters=None,
         decode_coords=None,
@@ -87,7 +87,7 @@ def open_dataset(
             filename_or_obj,
             *,
             drop_variables=None,
-            mask_and_scale=None,
+            mask_and_scale=True,
             decode_times=True,
             concat_characters=None,
             decode_coords=None,

cfapyx/creator.py

@@ -532,10 +532,14 @@ def _accumulate_attrs(self, attrs: dict, ncattrs: dict) -> dict:
                     if not np.array_equal(attrs[attr], ncattrs[attr]):
                         attrs[attr] = self.concat_msg
                     continue
-                if attrs[attr] != ncattrs[attr]:
+                try:
+                    if attrs.get(attr) != ncattrs.get(attr):
+                        attrs[attr] = self.concat_msg
+                    else:
+                        attrs[attr] = ncattrs[attr]
+                except ValueError:
+                    # Typically numpy array comparisons fail here.
                     attrs[attr] = self.concat_msg
-                else:
-                    attrs[attr] = ncattrs[attr]
         return attrs
 
 class CFAWriteMixin: