Merge pull request #125 from martius-lab/fkloss/push-uwtmwowyulso

Print warning when report generation failed.
martius-lab · Oct 28, 2024 · 04ad904 · 04ad904
2 parents 301302c + e6118bb
commit 04ad904
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/cluster_utils/server/report.py b/src/cluster_utils/server/report.py
@@ -22,7 +22,7 @@
 from . import data_analysis, distributions
 from .latex_utils import LatexFile
 from .optimizers import Optimizer
-from .utils import log_and_print, shorten_string
+from .utils import log_and_print, make_red, shorten_string
 
 
 def init_plotting():
@@ -313,7 +313,12 @@ def filename_gen(base_path):
             latex.produce_pdf(output_file)
             log_and_print(logger, f"Report saved at {output_file}")
         except Exception:
-            logging.warning("Could not generate PDF report", exc_info=True)
+            logging.warning("Failed to generate PDF report", exc_info=True)
+            print(
+                make_red(
+                    "Warning: Failed to generate PDF report.  See log for details."
+                )
+            )
 
 
 def distribution_plots(
@@ -494,4 +499,9 @@ def filename_gen(base_path: str | os.PathLike) -> Iterator[str]:
             latex.produce_pdf(output_file)
             logger.info("Saved report to %s", output_file)
         except Exception:
-            logger.warning("Could not generate PDF report", exc_info=True)
+            logger.warning("Failed to generate PDF report", exc_info=True)
+            print(
+                make_red(
+                    "Warning: Failed to generate PDF report.  See log for details."
+                )
+            )