Update app.py

app.py

@@ -14,6 +14,27 @@ logging.basicConfig(
 )
 logger = logging.getLogger(__name__)
 
+# Function to validate poppler_path
+def validate_poppler_path(poppler_path):
+    """
+    Validate if the provided poppler_path is valid and contains pdftoppm.
+    Args:
+        poppler_path (str): Path to poppler binaries.
+    Returns:
+        str or None: Valid poppler_path or None if invalid.
+    """
+    if not poppler_path:
+        return None
+    if not os.path.isdir(poppler_path):
+        logger.error("Invalid poppler_path: %s is not a directory", poppler_path)
+        return None
+    pdftoppm_path = os.path.join(poppler_path, "pdftoppm" + (".exe" if os.name == "nt" else ""))
+    if not os.path.isfile(pdftoppm_path):
+        logger.error("pdftoppm not found in poppler_path: %s", poppler_path)
+        return None
+    logger.debug("Valid poppler_path: %s", poppler_path)
+    return poppler_path
+
 # Function to calculate materials based on blueprint dimensions
 def calculate_materials_from_dimensions(wall_area, foundation_area):
     """
@@ -86,6 +107,10 @@ def process_blueprint(pdf_path, blueprint_width_m=27, blueprint_height_m=9.78, p
         logger.error("Invalid blueprint dimensions: width=%s, height=%s", blueprint_width_m, blueprint_height_m)
         return {"error": "Blueprint width and height must be positive"}
 
+    # Validate poppler_path
+    poppler_path = validate_poppler_path(poppler_path)
+    logger.debug("Using poppler_path: %s", poppler_path)
+
     try:
         # Convert PDF to images
         logger.debug("Converting PDF to image with poppler_path=%s", poppler_path)
@@ -199,7 +224,7 @@ def process_blueprint(pdf_path, blueprint_width_m=27, blueprint_height_m=9.78, p
                 "- Windows: Download from https://github.com/oschwartz10612/poppler-windows and add to PATH.\n"
                 "- Linux: Run 'sudo apt-get install poppler-utils'.\n"
                 "- Mac: Run 'brew install poppler'.\n"
-                "Alternatively, specify poppler_path in the input (e.g., C:/poppler/bin)."
+                "Alternatively, specify a valid poppler_path in the input (e.g., C:/poppler/bin)."
             )
         logger.error("Processing failed: %s", error_msg)
         return {"error": f"Failed to process PDF: {error_msg}"}
@@ -217,7 +242,7 @@ interface = gr.Interface(
     title="Blueprint Material Estimator",
     description=(
         "Upload a PDF containing a blueprint to estimate construction materials. "
-        "Specify blueprint dimensions and optionally provide the path to poppler binaries if not in PATH."
+        "Specify blueprint dimensions and provide the path to poppler binaries if not in PATH."
     )
 )