Merge branch 'master' into enhance-intan

Author: zm711

.gitignore

@@ -15,6 +15,8 @@
 .idea
 *.swp
 *.swo
+.vscode
+
 
 # Compiled source #
 ###################
@@ -56,6 +58,7 @@ cover
 .directory
 .gdb_history
 .DS_Store?
+.DS_Store
 ehthumbs.db
 Icon?
 Thumbs.db

neo/core/analogsignal.py

@@ -294,13 +294,7 @@ def __repr__(self):
         """
         Returns a string representing the :class:`AnalogSignal`.
         """
-        return "<%s(%s, [%s, %s], sampling rate: %s)>" % (
-            self.__class__.__name__,
-            super().__repr__(),
-            self.t_start,
-            self.t_stop,
-            self.sampling_rate,
-        )
+        return f"<{self.__class__.__name__}({super().__repr__()}, [{self.t_start}, {self.t_stop}], sampling rate: {self.sampling_rate})>"
 
     def __getitem__(self, i):
         """
@@ -324,7 +318,7 @@ def __getitem__(self, i):
                     raise NotImplementedError("Arrays not yet supported")  # in the general case, would need to return
                     #  IrregularlySampledSignal(Array)
                 else:
-                    raise TypeError("%s not supported" % type(j))
+                    raise TypeError(f"{type(j)} not supported")
                 if isinstance(k, (int, np.integer)):
                     obj = obj.reshape(-1, 1)
                 elif k is None:
@@ -473,7 +467,7 @@ def _check_consistency(self, other):
         if isinstance(other, AnalogSignal):
             for attr in "t_start", "sampling_rate":
                 if getattr(self, attr) != getattr(other, attr):
-                    raise ValueError("Inconsistent values of %s" % attr)  # how to handle name and annotations?
+                    raise ValueError(f"Inconsistent values of {attr}")  # how to handle name and annotations?
 
     def _repr_pretty_(self, pp, cycle):
         """

neo/core/baseneo.py

@@ -45,7 +45,9 @@ def _check_annotations(value):
         if not issubclass(value.dtype.type, ALLOWED_ANNOTATION_TYPES):
             raise ValueError(f"Invalid annotation. NumPy arrays with dtype {value.dtype.type}" f"are not allowed")
     elif isinstance(value, dict):
-        for element in value.values():
+        for key, element in value.items():
+            if not isinstance(key, str):
+                raise TypeError(f"Annotations keys must be strings not of type {type(key)}")
             _check_annotations(element)
     elif isinstance(value, (list, tuple)):
         for element in value:

neo/core/basesignal.py

@@ -237,7 +237,7 @@ def merge(self, other):
         for attr in self._necessary_attrs:
             if "signal" != attr[0]:
                 if getattr(self, attr[0], None) != getattr(other, attr[0], None):
-                    raise MergeError("Cannot merge these two signals as the %s differ." % attr[0])
+                    raise MergeError(f"Cannot merge these two signals as the {attr[0]} differ.")
 
         if self.segment != other.segment:
             raise MergeError("Cannot merge these two signals as they belong to different segments.")

neo/core/container.py

@@ -577,6 +577,6 @@ def _repr_pretty_(self, pp, cycle):
             pp.text(f"# {container} (N={objs})")
             for i, obj in enumerate(objs):
                 pp.breakable()
-                pp.text("%s: " % i)
+                pp.text(f"{i}: ")
                 with pp.indent(3):
                     pp.pretty(obj)

neo/core/dataobject.py

@@ -400,7 +400,7 @@ def __setitem__(self, key, value):
     def update(self, *args, **kwargs):
         if args:
             if len(args) > 1:
-                raise TypeError("update expected at most 1 arguments, " "got %d" % len(args))
+                raise TypeError("update expected at most 1 arguments, " f"got {len(args)}")
             other = dict(args[0])
             for key in other:
                 self[key] = other[key]

neo/core/epoch.py

@@ -155,7 +155,7 @@ def __new__(
         # this approach is much faster than comparing the
         # reference dimensionality
         if len(dim) != 1 or list(dim.values())[0] != 1 or not isinstance(list(dim.keys())[0], pq.UnitTime):
-            ValueError("Unit %s has dimensions %s, not [time]" % (units, dim.simplified))
+            ValueError(f"Unit {units} has dimensions {dim.simplified}, not [time]")
 
         obj = pq.Quantity.__new__(cls, times, units=dim)
         obj._labels = labels
@@ -227,8 +227,7 @@ def __repr__(self):
         """
 
         objs = [
-            "%s@%s for %s" % (label, str(time), str(dur))
-            for label, time, dur in zip(self.labels, self.times, self.durations)
+            f"{label}@{str(time)} for {str(dur)}" for label, time, dur in zip(self.labels, self.times, self.durations)
         ]
         return "<Epoch: %s>" % ", ".join(objs)
 

neo/core/event.py

@@ -194,7 +194,7 @@ def __repr__(self):
         Returns a string representing the :class:`Event`.
         """
 
-        objs = ["%s@%s" % (label, str(time)) for label, time in zip(self.labels, self.times)]
+        objs = [f"{label}@{str(time)}" for label, time in zip(self.labels, self.times)]
         return "<Event: %s>" % ", ".join(objs)
 
     def _repr_pretty_(self, pp, cycle):

neo/core/imagesequence.py

@@ -227,7 +227,7 @@ def _check_consistency(self, other):
         if isinstance(other, ImageSequence):
             for attr in ("sampling_rate", "spatial_scale", "t_start"):
                 if getattr(self, attr) != getattr(other, attr):
-                    raise ValueError("Inconsistent values of %s" % attr)
+                    raise ValueError(f"Inconsistent values of {attr}")
 
     # t_start attribute is handled as a property so type checking can be done
     @property

neo/core/irregularlysampledsignal.py

@@ -272,7 +272,7 @@ def __getitem__(self, i):
                 elif isinstance(j, np.ndarray):
                     raise NotImplementedError("Arrays not yet supported")
                 else:
-                    raise TypeError("%s not supported" % type(j))
+                    raise TypeError(f"{type(j)} not supported")
                 if isinstance(k, (int, np.integer)):
                     obj = obj.reshape(-1, 1)
                 obj.array_annotations = deepcopy(self.array_annotations_at_index(k))
@@ -600,7 +600,7 @@ def concatenate(self, other, allow_overlap=False):
         for attr in self._necessary_attrs:
             if not (attr[0] in ["signal", "times", "t_start", "t_stop", "times"]):
                 if getattr(self, attr[0], None) != getattr(other, attr[0], None):
-                    raise MergeError("Cannot concatenate these two signals as the %s differ." % attr[0])
+                    raise MergeError(f"Cannot concatenate these two signals as the {attr[0]} differ.")
 
         if hasattr(self, "lazy_shape"):
             if hasattr(other, "lazy_shape"):