Update app.py

app.py

@@ -10,21 +10,15 @@ import gradio as gr
 from transformers import GPT2Tokenizer, GPT2LMHeadModel
 import torch
 
-# --- Importar la librería de Pexels ---
 from pexels_api import API 
+from keybert import KeyBERT 
 
-# --- Importar KeyBERT para extracción de palabras clave ---
-from keybert import KeyBERT # Asegúrate de tener 'keybert' y 'sentence-transformers' en requirements.txt
-
-# --- Configuración de Logging ---
 logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
 logger = logging.getLogger(__name__)
 
-# --- Clave API de Pexels ---
 PEXELS_API_KEY = os.environ.get("PEXELS_API_KEY") 
 pexels_api = API(PEXELS_API_KEY) 
 
-# --- Inicialización de Tokenizer y Modelo GPT-2 ---
 MODEL_NAME = "gpt2-small" 
 try:
     tokenizer = GPT2Tokenizer.from_pretrained(MODEL_NAME)
@@ -37,18 +31,13 @@ except Exception as e:
     tokenizer = None
     model = None
 
-# --- Inicialización del modelo KeyBERT ---
 try:
-    # Se descarga la primera vez. 'multi-qa-MiniLM-L6-cos-v1' es un buen modelo multilingüe pequeño.
-    # Considera 'distiluse-base-multilingual-cased-v2' para mejor calidad multilingüe si hay suficiente RAM.
     kw_model = KeyBERT('multi-qa-MiniLM-L6-cos-v1') 
     logger.info("Modelo KeyBERT cargado exitosamente.")
 except Exception as e:
     logger.error(f"Error al cargar el modelo KeyBERT: {e}. La búsqueda de videos será menos precisa.", exc_info=True)
     kw_model = None
 
-# --- Funciones de Utilidad ---
-
 def generate_script(prompt, max_length=250): 
     if not tokenizer or not model:
         logger.error("Modelo GPT-2 no disponible para generar guion.")
@@ -89,19 +78,18 @@ async def text_to_speech(text, voice="es-ES-ElviraNeural", output_path="voz.mp3"
         logger.error(f"Error al generar audio TTS: {e}")
         raise
 
-def download_video_file(url, temp_dir): # Renombrada y mejorada para descargar a un dir temporal
+def download_video_file(url, temp_dir): 
     if not url:
         return None
     
-    # Intentar obtener el nombre del archivo de la URL o generar uno
     file_name = url.split('/')[-1].split('?')[0]
-    if not file_name.endswith('.mp4'): # Asegurar extensión correcta
+    if not file_name.endswith('.mp4'): 
         file_name = f"video_temp_{os.getpid()}_{datetime.now().strftime('%f')}.mp4"
     
     output_path = os.path.join(temp_dir, file_name)
     logger.info(f"Intentando descargar video de: {url} a {output_path}")
     try:
-        response = requests.get(url, stream=True, timeout=30) # Aumentar timeout por videos grandes
+        response = requests.get(url, stream=True, timeout=30) 
         response.raise_for_status()
 
         with open(output_path, 'wb') as f:
@@ -130,57 +118,40 @@ def loop_audio_to_length(audio_clip, target_duration):
     concatenated = concatenate_videoclips(audios)
     return concatenated.subclip(0, target_duration)
 
-# --- NUEVA LÓGICA: Extracción de palabras clave visuales del guion ---
 def extract_visual_keywords_from_script(script_text, max_keywords_per_segment=2):
-    """
-    Extrae palabras clave visuales de un script dividiéndolo en segmentos.
-    Retorna una lista de términos de búsqueda únicos.
-    """
     if not kw_model:
         logger.warning("Modelo KeyBERT no disponible. La extracción de palabras clave será limitada.")
-        # Fallback si KeyBERT no carga: usar las primeras palabras del guion
         return [script_text.split('.')[0].strip().replace(" ", "+")] if script_text.strip() else []
 
     logger.info("Extrayendo palabras clave visuales del guion.")
     
-    # Dividir el guion en frases o párrafos para un mejor análisis contextual
-    # Puedes ajustar esto, por ejemplo, split('.') si prefieres por frases.
     segments = [s.strip() for s in script_text.split('\n') if s.strip()]
-    if not segments: # Si no se puede dividir, usar el texto completo
+    if not segments: 
         segments = [script_text]
 
-    all_keywords = set() # Usar un set para evitar duplicados
+    all_keywords = set() 
     
     for segment in segments:
         if not segment: continue
         try:
-            # keyphrase_ngram_range=(1,2) busca palabras sueltas o frases de dos palabras
-            # top_n=max_keywords_per_segment es cuántas palabras clave por segmento
-            # use_mmr y diversity ayudan a obtener palabras clave más variadas.
             keywords_with_scores = kw_model.extract_keywords(
                 segment,
                 keyphrase_ngram_range=(1, 2), 
-                stop_words='spanish', # ¡IMPORTANTE! Usar stop words en español
+                stop_words='spanish', 
                 top_n=max_keywords_per_segment,
                 use_mmr=True, 
                 diversity=0.7 
             )
             for kw, score in keywords_with_scores:
-                # Convertir a formato de búsqueda de URL (espacios por +)
                 all_keywords.add(kw.replace(" ", "+"))
         except Exception as e:
             logger.warning(f"Error al extraer palabras clave de un segmento: {e}")
-            # En caso de error en un segmento, intentar con el segmento completo como palabra clave simple
             all_keywords.add(segment.split(' ')[0].strip().replace(" ", "+"))
 
     logger.info(f"Palabras clave visuales extraídas del guion: {list(all_keywords)}")
-    return list(all_keywords) # Devolver como lista para iterar
+    return list(all_keywords) 
 
-# --- Búsqueda de videos en Pexels ---
 def search_pexels_videos(query_list, num_videos_per_query=5, min_duration_sec=7):
-    """
-    Busca videos en Pexels basados en una lista de queries y devuelve URLs de descarga.
-    """
     if not PEXELS_API_KEY:
         logger.error("ERROR: PEXELS_API_KEY no está configurada. No se pueden buscar videos en Pexels.")
         raise ValueError("PEXELS_API_KEY no está configurada. Por favor, configúrala como un secreto en Hugging Face Spaces.")
@@ -189,12 +160,11 @@ def search_pexels_videos(query_list, num_videos_per_query=5, min_duration_sec=7)
         logger.warning("Lista de queries para Pexels vacía. No se buscarán videos.")
         return []
 
-    all_video_urls = set() # Usar un set para evitar URLs duplicadas
+    all_video_urls = set() 
     
     for query in query_list:
         logger.info(f"Buscando {num_videos_per_query} videos en Pexels para la query: '{query}'")
         try:
-            # Pedimos más de los que necesitamos por query por si algunos no cumplen duración
             pexels_api.search_videos(query, per_page=num_videos_per_query * 2) 
             videos = pexels_api.get_videos()
 
@@ -204,7 +174,6 @@ def search_pexels_videos(query_list, num_videos_per_query=5, min_duration_sec=7)
 
             for video in videos:
                 if video.get('duration') and video['duration'] < min_duration_sec:
-                    # logger.info(f"Saltando video {video['id']} de Pexels por duración ({video['duration']}s < {min_duration_sec}s).")
                     continue
 
                 best_quality_url = None
@@ -222,15 +191,11 @@ def search_pexels_videos(query_list, num_videos_per_query=5, min_duration_sec=7)
 
         except Exception as e:
             logger.error(f"Error al buscar videos en Pexels para '{query}': {e}", exc_info=True)
-            # No lanzamos la excepción aquí, para que otras queries puedan seguir funcionando
-            # Pero el error se registra.
             
     final_urls = list(all_video_urls)
     logger.info(f"Total de {len(final_urls)} URLs de video únicas obtenidas de Pexels.")
     return final_urls
 
-# --- Función Principal de Creación de Video ---
-
 def crear_video(prompt_type, input_text, musica_url=None):
     logger.info(f"Iniciando creación de video. Tipo de prompt: {prompt_type}")
     guion = ""
@@ -247,40 +212,32 @@ def crear_video(prompt_type, input_text, musica_url=None):
         raise ValueError("El guion está vacío. No se puede proceder.")
 
     temp_files = [] 
-    downloaded_clip_paths = [] # Para los paths de los videos descargados
-    final_clips = [] # Para los objetos VideoFileClip ya procesados
+    downloaded_clip_paths = [] 
+    final_clips = [] 
 
-    # Directorio temporal para descargar videos
     temp_video_dir = tempfile.mkdtemp()
-    temp_files.append(temp_video_dir) # Añadir el directorio temporal para su limpieza
+    temp_files.append(temp_video_dir) 
 
     try:
-        # 1. Generar audio TTS
         voz_archivo = os.path.join(tempfile.gettempdir(), f"voz_temp_{os.getpid()}.mp3")
         temp_files.append(voz_archivo)
         asyncio.run(text_to_speech(guion, output_path=voz_archivo))
         audio_tts = AudioFileClip(voz_archivo)
 
-        # 2. Extracción de palabras clave y búsqueda masiva en Pexels
-        # Dividimos el guion en segmentos y extraemos palabras clave de cada uno
-        # Esto genera una lista de queries para Pexels, como ['misterios+china', 'muralla+china', ...]
         search_queries_for_pexels = extract_visual_keywords_from_script(guion, max_keywords_per_segment=2)
 
-        # Si no se pudo extraer ninguna palabra clave útil, lanzar error.
         if not search_queries_for_pexels:
             raise ValueError("No se pudieron extraer palabras clave visuales del guion para buscar videos.")
 
-        # Buscar y obtener URLs de muchos videos de Pexels (e.g., 5 videos por cada palabra clave)
         pexels_video_urls_found = search_pexels_videos(search_queries_for_pexels, num_videos_per_query=5, min_duration_sec=7) 
 
         if not pexels_video_urls_found:
             logger.error("Pexels no devolvió ningún video para las palabras clave extraídas.")
             raise ValueError(f"Pexels no encontró videos adecuados para el guion. Intenta con otro tema o guion más descriptivo. Palabras clave usadas: {search_queries_for_pexels}")
 
-        # 3. Descargar todos los videos encontrados de Pexels
         logger.info(f"Descargando {len(pexels_video_urls_found)} videos de Pexels...")
         for url in pexels_video_urls_found:
-            video_path = download_video_file(url, temp_video_dir) # Descargar al directorio temporal
+            video_path = download_video_file(url, temp_video_dir) 
             if video_path:
                 downloaded_clip_paths.append(video_path)
             else:
@@ -290,53 +247,45 @@ def crear_video(prompt_type, input_text, musica_url=None):
             logger.error("No se pudo descargar ningún video válido de Pexels.")
             raise ValueError("No se pudo descargar ningún video válido de Pexels. Revisa la conexión o la calidad de las URLs.")
 
-        # 4. Cargar y procesar los clips descargados
-        total_desired_video_duration = audio_tts.duration * 1.2 # Intentar tener 20% más de video que audio
+        total_desired_video_duration = audio_tts.duration * 1.2 
         current_video_clips_duration = 0
         
         for path in downloaded_clip_paths:
             try:
                 clip = VideoFileClip(path)
-                # Opcional: Si quieres clips cortos y variados para transiciones rápidas
-                clip_duration = min(clip.duration, 10) # Limitar a un máximo de 10 segundos por clip
-                if clip_duration > 1: # Asegurar que el clip sea lo suficientemente largo
+                clip_duration = min(clip.duration, 10) 
+                if clip_duration > 1: 
                     final_clips.append(clip.subclip(0, clip_duration))
                     current_video_clips_duration += clip_duration
                     if current_video_clips_duration >= total_desired_video_duration:
-                        break # Ya tenemos suficiente metraje
+                        break 
                 else:
                     logger.warning(f"Clip de video muy corto ({clip_duration:.2f}s) de {path}, omitiendo.")
                     clip.close()
             except Exception as e:
                 logger.warning(f"No se pudo cargar o procesar el clip de video {path}: {e}. Omitiendo.")
-                if 'clip' in locals() and clip: clip.close() # Asegurarse de cerrar si hay un error
+                if 'clip' in locals() and clip: clip.close() 
         
         if not final_clips:
             logger.error("No se pudo obtener ningún clip de video usable después de la descarga y procesamiento.")
             raise ValueError("No se pudieron procesar los videos descargados de Pexels.")
 
-
-        # 5. Concatenar videos y ajustar duración
         video_base = concatenate_videoclips(final_clips, method="compose")
         logger.info(f"Video base concatenado, duración: {video_base.duration:.2f}s")
 
-        # Asegurarse de que el video base sea al menos tan largo como el audio TTS
-        # Si es más corto, loopearlo.
         if video_base.duration < audio_tts.duration:
             logger.info(f"Duración del video ({video_base.duration:.2f}s) es menor que la del audio TTS ({audio_tts.duration:.2f}s). Repitiendo video.")
             num_repeats = int(audio_tts.duration / video_base.duration) + 1
             repeated_clips = [video_base] * num_repeats
             video_base = concatenate_videoclips(repeated_clips, method="compose") 
         
-        # El video final siempre tendrá la duración del audio TTS (o la mezcla de audio)
         final_video_duration = audio_tts.duration
 
-        # 6. Música de fondo en loop
         mezcla_audio = audio_tts
         if musica_url and musica_url.strip():
-            musica_path = download_video_file(musica_url, temp_video_dir) # Usar la misma función de descarga
+            musica_path = download_video_file(musica_url, temp_video_dir) 
             if musica_path:
-                temp_files.append(musica_path) # Añadir el path del archivo de música a limpiar
+                temp_files.append(musica_path) 
                 try:
                     musica_audio = AudioFileClip(musica_path)
                     musica_loop = loop_audio_to_length(musica_audio, final_video_duration)
@@ -347,11 +296,9 @@ def crear_video(prompt_type, input_text, musica_url=None):
             else:
                 logger.warning(f"No se pudo descargar la música de {musica_url}. Se usará solo la voz.")
         
-        # 7. Asignar audio al video y ajustar duración
         video_final = video_base.set_audio(mezcla_audio).subclip(0, final_video_duration)
         logger.info(f"Video final configurado con audio. Duración final: {video_final.duration:.2f}s")
 
-        # 8. Guardar video final
         output_dir = "generated_videos"
         os.makedirs(output_dir, exist_ok=True)
         output_path = os.path.join(output_dir, f"video_output_{datetime.now().strftime('%Y%m%d_%H%M%S')}.mp4")
@@ -365,34 +312,26 @@ def crear_video(prompt_type, input_text, musica_url=None):
         logger.error(f"Error general en la creación del video: {e}", exc_info=True)
         raise e
     finally:
-        # 9. Limpiar archivos y directorios temporales
         for f in temp_files:
             if os.path.exists(f):
                 if os.path.isdir(f):
-                    try:
-                        import shutil
-                        shutil.rmtree(f)
-                        logger.info(f"Directorio temporal eliminado: {f}")
-                    except Exception as e:
-                        logger.warning(f"No se pudo eliminar el directorio temporal {f}: {e}")
+                    import shutil
+                    shutil.rmtree(f)
+                    logger.info(f"Directorio temporal eliminado: {f}")
                 else:
-                    try:
-                        os.remove(f)
-                        logger.info(f"Archivo temporal eliminado: {f}")
-                    except Exception as e:
-                        logger.warning(f"No se pudo eliminar el archivo temporal {f}: {e}")
+                    os.remove(f)
+                    logger.info(f"Archivo temporal eliminado: {f}")
         
-        # Asegurarse de cerrar todos los objetos MoviePy VideoFileClip y AudioFileClip
-        for clip in final_clips: # Iterar sobre los clips finales que se usaron
-            if clip and not clip.is_playing: # Solo cerrar si no está ya cerrado
+        for clip in final_clips: 
+            if clip and hasattr(clip, 'close') and not clip.is_playing:
                 clip.close()
-        if 'audio_tts' in locals() and audio_tts and not audio_tts.is_playing:
+        if 'audio_tts' in locals() and audio_tts and hasattr(audio_tts, 'close') and not audio_tts.is_playing:
             audio_tts.close()
-        if 'musica_audio' in locals() and musica_audio and not musica_audio.is_playing:
+        if 'musica_audio' in locals() and musica_audio and hasattr(musica_audio, 'close') and not musica_audio.is_playing:
             musica_audio.close()
-        if 'video_final' in locals() and video_final and not video_final.is_playing:
+        if 'video_final' in locals() and video_final and hasattr(video_final, 'close') and not video_final.is_playing:
             video_final.close()
-        if 'video_base' in locals() and video_base and not video_base.is_playing:
+        if 'video_base' in locals() and video_base and hasattr(video_base, 'close') and not video_base.is_playing:
             video_base.close()
 
 
@@ -423,7 +362,6 @@ def run_app(prompt_type, prompt_ia, prompt_manual, musica_url):
         logger.error(f"Error inesperado al ejecutar la aplicación: {e}", exc_info=True)
         return None, gr.update(value=f"Ocurrió un error grave: {e}. Por favor, inténtalo de nuevo.", text_color="red")
 
-# --- Interfaz de Gradio ---
 with gr.Blocks() as app:
     gr.Markdown("""
     ### 🎬 Generador de Video Inteligente con Pexels 🚀