fix: improve exception safety and logging

tasi/base.py

@@ -29,7 +29,7 @@ def att(
 
         try:
             len(timestamps)
-        except:
+        except TypeError:
             timestamps = [timestamps]
 
         if self.index.nlevels == 1:
@@ -184,7 +184,7 @@ def atid(
         """
         try:
             len(ids)
-        except BaseException:
+        except TypeError:
             ids = [ids]
 
         if attributes is None:

tasi/indexing.py

@@ -37,10 +37,14 @@ def __getitem__(self, key):
                 # if the levels do not match, try wrapping the key in a list
                 try:
                     df = super().__getitem__([key])
-                except Exception as err:
-                    # this will not work for slices. Just ignore this.
-                    print(err)
-                    pass
+                except (KeyError, ValueError, TypeError) as err:
+                    # this will not work for slices. Log and ignore gracefully.
+                    import logging
+                    logging.getLogger(__name__).debug(
+                        "Dialog LocLocator index fallback failed for key %r: %s",
+                        key,
+                        err,
+                    )
             return self.obj._ensure_correct_type(df, key)
         except AttributeError:
             # if the result is no pandas object anymore. This occurs if the user selects specific table cell using any

tasi/io/public/base.py

@@ -49,7 +49,7 @@ def flatten_dataframe_columns(df: pd.DataFrame, max_levels=1):
     if df.columns.nlevels > max_levels:
         try:
             df = df.droplevel(level=1, axis=1)
-        except:
+        except (KeyError, ValueError, IndexError):
             df = df.droplevel(level=1)
 
     return df

tasi/plotting/wms.py

@@ -104,10 +104,10 @@ def get_tile(self, x1, y1, x2, y2):
         try:
             fp = _io.BytesIO(tile)
             return _Image.open(fp)
-        except BaseException:
+        except (OSError, IOError, ValueError) as err:
             raise RuntimeError(
-                "Failed to decode data for {} - @ {} extent".format(
-                    self.name, [x1, y1, x2, y2]
+                "Failed to decode data for {} - @ {} extent: {}".format(
+                    self.name, [x1, y1, x2, y2], err
                 )
             )