Update app.py

app.py

@@ -9,13 +9,12 @@ if not ASR_API_URL or not AUTH_TOKEN:
     print("⚠️  ASR_API_URL or AUTH_TOKEN is not set; API calls will fail.")
 
 # ---------- Core Transcription Function ----------
-def transcribe_audio(file_path: str | None): # Added None type hint for clarity on clearing
-    # Handle case where audio is cleared (input might be None)
+def transcribe_audio(file_path: str | None):
     if file_path is None:
-        return "Audio cleared.", "", None # Return default/empty values
+        return "Audio cleared.", "", None
 
     if not ASR_API_URL or not AUTH_TOKEN:
-        return "❌ Error: ASR_API_URL or AUTH_TOKEN is not set.", "", file_path # Keep file path on error
+        return "❌ Error: ASR_API_URL or AUTH_TOKEN is not set.", "", file_path
 
     headers = {
         "accept": "application/json",
@@ -25,14 +24,13 @@ def transcribe_audio(file_path: str | None): # Added None type hint for clarity
     start = time.time()
     try:
         with open(file_path, "rb") as f:
-            # Ensure the filename is correctly extracted, especially for temp files
             file_name = os.path.basename(file_path)
-            files = {"file": (file_name, f, "audio/mpeg")} # Use common mpeg, adjust if specific format needed
-            resp = requests.post(ASR_API_URL, headers=headers, files=files, timeout=120) # Increased timeout just in case
+            # Adjusted mime type slightly - more general
+            files = {"file": (file_name, f, "audio/wav")} # Use audio/wav or audio/mpeg as appropriate for your API
+            resp = requests.post(ASR_API_URL, headers=headers, files=files, timeout=120)
     except requests.exceptions.Timeout:
          return f"❌ Error: Request timed out after 120 seconds.", "", file_path
     except Exception as e:
-        # Provide more specific error context if possible
         return f"❌ Error during API call or file handling: {e}", "", file_path
 
     elapsed = time.time() - start
@@ -40,13 +38,11 @@ def transcribe_audio(file_path: str | None): # Added None type hint for clarity
         try:
             data = resp.json()
             text = data.get("transcription", "No transcription returned.")
-            # Use the 'time' field from response if available, otherwise use measured elapsed time
             processing_time = data.get('time', elapsed)
-            return text, f"{processing_time:.2f} s", file_path # Return filepath to keep it in the input widget if needed
+            return text, f"{processing_time:.2f} s", file_path
         except requests.exceptions.JSONDecodeError:
             return f"❌ Error: Could not decode JSON response. Status: {resp.status_code}, Response: {resp.text}", "", file_path
     else:
-        # Return error details from the response
         return f"❌ Error: API returned status {resp.status_code}. Response: {resp.text}", "", file_path
 
 
@@ -68,7 +64,6 @@ custom_css = f"""
 
 # ---------- UI ----------
 with gr.Blocks(css=custom_css, title="Gooya ASR v1.4") as demo:
-    # Optional: Add a title using Markdown or HTML
     gr.Markdown("# Gooya ASR v1.4 Transcription", elem_id="gooya-title")
     with gr.Row():
         with gr.Column():
@@ -81,14 +76,14 @@ with gr.Blocks(css=custom_css, title="Gooya ASR v1.4") as demo:
             processing_time_tb = gr.Textbox(
                 label="⏱️ Processing Time",
                 interactive=False,
-                elem_classes="gooya-badge", # Use elem_classes for multiple classes
+                elem_classes="gooya-badge",
             )
             transcription_tb = gr.Textbox(
                 label="📝 Transcription",
                 lines=5,
                 show_copy_button=True,
                 placeholder="The transcription will appear here...",
-                elem_id="gooya-textbox", # elem_id should be unique if used
+                elem_id="gooya-textbox",
             )
 
     with gr.Row():
@@ -106,25 +101,23 @@ with gr.Blocks(css=custom_css, title="Gooya ASR v1.4") as demo:
     )
 
     # ---------- Callbacks ----------
-    # Update outputs to potentially include audio_input if you want to keep it on error
     btn_transcribe.click(
         fn=transcribe_audio,
         inputs=[audio_input],
-        outputs=[transcription_tb, processing_time_tb, audio_input], # Keep audio input displayed
+        outputs=[transcription_tb, processing_time_tb, audio_input],
     )
 
-    # Clear function
     def clear_all():
-        return "", "", None # Clears transcription, time, and audio input
+        return "", "", None
 
     btn_clear.click(
-        fn=clear_all, # Use a named function for clarity
+        fn=clear_all,
         inputs=None,
         outputs=[transcription_tb, processing_time_tb, audio_input],
     )
 
 # ---------- Launch ----------
 if __name__ == "__main__":
-    # Set share=True to generate a public link when localhost is not accessible
-    # This is necessary in environments like Docker containers or cloud platforms (e.g., HF Spaces)
-    demo.queue().launch(debug=True, share=True) # <-- Changed share=False to share=True
+    # On Hugging Face Spaces, share=True is not needed and causes a warning.
+    # debug=True might also cause issues in some restricted environments, remove if necessary.
+    demo.queue().launch(debug=False) # <-- Reverted share=True, optionally removed debug=True