Update app.py

app.py

@@ -9,6 +9,7 @@ from typing import Optional, Tuple, List, Union
 import logging
 from functools import partial
 import shutil
+import sys
 
 # Configure logging
 logging.basicConfig(
@@ -18,20 +19,50 @@ logging.basicConfig(
 logger = logging.getLogger(__name__)
 
 class ImageToDxfConverter:
-    def __init__(self, executable_path: str = "./SimpleImageToDxfHavePass"):
+    def __init__(self):
         """Initialize the converter with configuration."""
-        self.executable_path = Path(executable_path)
+        self.executable_path = self._find_executable()
         self.hf_token = os.getenv("HF_TOKEN")
         self.repo_id = "ArrcttacsrjksX/ImageToAutocadData"
         
-        # Validate executable exists
-        if not self.executable_path.exists():
-            raise FileNotFoundError(f"Executable not found: {executable_path}")
-        
         # Validate HF token
         if not self.hf_token:
             logger.warning("HF_TOKEN environment variable not set")
     
+    def _find_executable(self) -> Optional[Path]:
+        """Find the converter executable in various possible locations."""
+        possible_paths = [
+            Path("./SimpleImageToDxfHavePass"),
+            Path("./SimpleImageToDxfHavePass.exe"),
+            Path("/app/SimpleImageToDxfHavePass"),  # Common Docker path
+            Path(os.path.dirname(os.path.abspath(__file__))) / "SimpleImageToDxfHavePass",
+            Path.home() / "SimpleImageToDxfHavePass"
+        ]
+
+        # Add system PATH locations
+        if sys.platform == "win32":
+            if "PATH" in os.environ:
+                for path in os.environ["PATH"].split(os.pathsep):
+                    possible_paths.append(Path(path) / "SimpleImageToDxfHavePass.exe")
+        else:
+            for path in ["/usr/local/bin", "/usr/bin", "/bin"]:
+                possible_paths.append(Path(path) / "SimpleImageToDxfHavePass")
+
+        for path in possible_paths:
+            if path.exists():
+                try:
+                    # Make executable on Unix-like systems
+                    if sys.platform != "win32":
+                        path.chmod(path.stat().st_mode | 0o111)
+                    logger.info(f"Found executable at: {path}")
+                    return path
+                except Exception as e:
+                    logger.warning(f"Found executable at {path} but couldn't set permissions: {e}")
+                    continue
+
+        logger.error("SimpleImageToDxfHavePass executable not found in any standard location")
+        return None
+
     def _ensure_directory(self, path: Union[str, Path]) -> Path:
         """Ensure directory exists and return Path object."""
         path = Path(path)
@@ -49,7 +80,18 @@ class ImageToDxfConverter:
 
     def _execute_conversion(self, command_args: List[str]) -> subprocess.CompletedProcess:
         """Execute the conversion command with proper error handling."""
+        if not self.executable_path:
+            raise RuntimeError(
+                "SimpleImageToDxfHavePass executable not found. Please ensure it's installed "
+                "and available in your system PATH or in the same directory as the script."
+            )
+
         try:
+            # Add executable verification step
+            if not os.access(str(self.executable_path), os.X_OK) and sys.platform != "win32":
+                logger.warning(f"Setting executable permissions for {self.executable_path}")
+                os.chmod(str(self.executable_path), 0o755)
+
             result = subprocess.run(
                 command_args,
                 check=True,
@@ -62,6 +104,10 @@ class ImageToDxfConverter:
             error_msg = f"Conversion failed: {e.stderr}"
             logger.error(error_msg)
             raise RuntimeError(error_msg)
+        except Exception as e:
+            error_msg = f"Unexpected error during conversion: {str(e)}"
+            logger.error(error_msg)
+            raise RuntimeError(error_msg)
 
     def _upload_to_huggingface(self, 
                              file_path: Path, 
@@ -97,6 +143,12 @@ class ImageToDxfConverter:
             if not image_path:
                 raise ValueError("No image provided")
 
+            if not self.executable_path:
+                raise RuntimeError(
+                    "SimpleImageToDxfHavePass executable not found. Please ensure it's installed "
+                    "and available in your system PATH or in the same directory as the script."
+                )
+
             # Setup output directory
             output_dir = self._ensure_directory(output_folder if output_folder else tempfile.mkdtemp())
             timestamp = datetime.datetime.now().strftime("%Y-%m-%d_%H-%M-%S")
@@ -156,19 +208,35 @@ class ImageToDxfConverter:
 
         except Exception as e:
             logger.error(f"Conversion failed: {str(e)}")
-            return None, None, []
+            return str(e), None, []
 
 def create_gradio_interface():
     """Create and configure the Gradio interface."""
     converter = ImageToDxfConverter()
     
+    # Early validation of executable
+    if not converter.executable_path:
+        logger.error("No valid executable found - interface will show error message")
+    
     with gr.Blocks(theme=gr.themes.Soft()) as demo:
         with gr.Tabs():
             with gr.TabItem("Image to DXF"):
-                gr.Markdown("""
-                # Image to DXF Converter
-                Convert your images to DXF format for CAD software.
-                """)
+                if not converter.executable_path:
+                    gr.Markdown("""
+                    # ⚠️ System Configuration Error
+                    
+                    The SimpleImageToDxfHavePass executable was not found. Please ensure:
+                    1. The executable is in the same directory as this script
+                    2. The executable has proper permissions
+                    3. The executable is available in your system PATH
+                    
+                    Check the logs for more details.
+                    """)
+                else:
+                    gr.Markdown("""
+                    # Image to DXF Converter
+                    Convert your images to DXF format for CAD software.
+                    """)
                 
                 with gr.Row():
                     with gr.Column(scale=2):
@@ -191,7 +259,8 @@ def create_gradio_interface():
                         convert_btn = gr.Button(
                             "Convert to DXF",
                             variant="primary",
-                            elem_id="convert_btn"
+                            elem_id="convert_btn",
+                            interactive=bool(converter.executable_path)
                         )
 
                 with gr.Row():
@@ -211,7 +280,7 @@ def create_gradio_interface():
                 error_output = gr.Textbox(
                     label="Status",
                     interactive=False,
-                    visible=False,
+                    visible=True,
                     elem_id="error_output"
                 )
 
@@ -233,13 +302,18 @@ def create_gradio_interface():
                 2. (Optional) Specify output folder
                 3. Toggle line detection if needed
                 4. Click Convert
+                
+                ### System Requirements:
+                - SimpleImageToDxfHavePass executable must be available
+                - Proper file permissions
+                - HF_TOKEN environment variable for Hugging Face integration
                 """)
 
         # Setup event handlers
         convert_btn.click(
             fn=converter.convert_image,
             inputs=[image_input, output_folder, use_lines_checkbox],
-            outputs=[dxf_output, debug_output, error_output],
+            outputs=[error_output, debug_output, dxf_output],
             api_name="convert"
         )
 
@@ -248,15 +322,9 @@ def create_gradio_interface():
 def main():
     """Main entry point with proper error handling."""
     try:
-        # Ensure executable permissions
-        executable = Path("./SimpleImageToDxfHavePass")
-        if executable.exists():
-            executable.chmod(0o755)
-        
         # Create and launch the interface
         demo = create_gradio_interface()
         demo.launch(
-            share=True,
             server_name="0.0.0.0",
             server_port=7860,
             show_error=True