Replaced print statement with logger.exception. Long term, might be nice to include it in the applications logger

Author: PaulJonasJost

src/petab_gui/views/simple_plot_view.py

@@ -1,3 +1,4 @@
+import logging
 from collections import defaultdict
 
 import petab.v1.C as PETAB_C
@@ -19,6 +20,8 @@
 
 from .utils import proxy_to_dataframe
 
+logger = logging.getLogger(__name__)
+
 
 class PlotWorkerSignals(QObject):
     finished = Signal(object)  # Emits final Figure
@@ -196,8 +199,8 @@ def _render_on_main_thread(self, payload):
                     fig = plt.gcf()
                     self._update_tabs(fig)
                     return
-                except Exception as e:
-                    print(f"Invalid Visualisation DF: {e}")
+                except Exception:
+                    logger.exception("Invalid Visualisation DF")
             # fallback to observable grouping
             plt.close("all")
             petab_vis.plot_without_vis_spec(
@@ -412,8 +415,8 @@ def plot_residuals(self):
                 axes=axes,
             )
             create_plot_tab(fig_res, self, "Residuals vs Simulation")
-        except ValueError as e:
-            print(f"Error plotting residuals: {e}")
+        except ValueError:
+            logger.exception("Error plotting residuals")
         fig_fit, axes_fit = plt.subplots(constrained_layout=False)
         fig_fit.subplots_adjust(left=0.05, right=0.98, bottom=0.05, top=0.98)
         plot_goodness_of_fit(