quick

app.py

@@ -1,10 +1,20 @@
 # app.py
 import gradio as gr
+import logging # Import logging
+import sys # Import sys to output to stderr
 # Removed imports: pandas, openpyxl, datetime, timedelta, os, tempfile
 
 # Import the processing function from main.py
 from main import process_files
 
+# --- Configure Logging --- (Configure before functions)
+log_format = '%(asctime)s - %(name)s - %(levelname)s - %(message)s'
+logging.basicConfig(level=logging.INFO, # Use INFO for general info, DEBUG for more detail
+                    format=log_format,
+                    handlers=[logging.StreamHandler(sys.stderr)]) # Output to stderr
+
+logger = logging.getLogger(__name__) # Get logger for this module
+
 def generate_report(file1_obj, file2_obj):
     """
     Gradio wrapper function to handle file uploads and call the main processing logic.
@@ -18,35 +28,40 @@ def generate_report(file1_obj, file2_obj):
     Raises:
         gr.Error: If file processing fails.
     """
+    logger.info("generate_report function called.")
     if file1_obj is None or file2_obj is None:
+        logger.error("Missing one or both input files.")
         raise gr.Error("Please upload both required files.")
 
     try:
         # Gradio provides temporary paths for uploaded files
         file1_path = file1_obj.name
         file2_path = file2_obj.name
+        logger.info(f"Input file 1 path: {file1_path}")
+        logger.info(f"Input file 2 path: {file2_path}")
 
-        # Define the output filename (can be customized if needed)
+        # Define the output filename
         output_filename = "generated_report.xlsx"
+        logger.info(f"Target output base filename: {output_filename}")
 
         # Call the core processing logic from main.py
-        print(f"Processing files: {file1_path}, {file2_path}") # Log input paths
+        logger.info(f"Calling process_files...")
         result_path = process_files(file1_path, file2_path, output_filename)
-        print(f"process_files returned: {result_path}") # Log result path
+        logger.info(f"process_files returned: {result_path}")
 
         if result_path:
+            logger.info(f"Report generation successful. Returning path: {result_path}")
             # Return the path of the generated file for Gradio to serve
             return result_path
         else:
             # If process_files returned None, it means an error occurred
-            raise gr.Error("Failed to generate the report. Check logs or input files.")
+            logger.error("process_files returned None, indicating failure.")
+            raise gr.Error("Failed to generate the report. Check application logs for details.")
 
     except Exception as e:
         # Catch any other unexpected errors during the wrapper execution
-        import traceback
-        print(f"Error in Gradio wrapper (generate_report): {e}")
-        print(traceback.format_exc())
-        raise gr.Error(f"An unexpected error occurred: {e}")
+        logger.exception("An unexpected error occurred in the Gradio wrapper (generate_report)") # Logs exception info automatically
+        raise gr.Error(f"An unexpected error occurred. Check application logs for details.") # User-friendly message
 
 
 # Create Gradio Interface

requirements.txt

@@ -1,3 +1,4 @@
 pandas>=1.3.0
 openpyxl>=3.0.0
+xlrd>=1.2.0
 gradio>=3.0