Update core/visual_engine.py

core/visual_engine.py

@@ -15,16 +15,18 @@ from moviepy.editor import (ImageClip, VideoFileClip, concatenate_videoclips, Te
                             CompositeVideoClip, AudioFileClip)
 import moviepy.video.fx.all as vfx
 
-try: # MONKEY PATCH
+try:
     if hasattr(Image, 'Resampling') and hasattr(Image.Resampling, 'LANCZOS'):
         if not hasattr(Image, 'ANTIALIAS'): Image.ANTIALIAS = Image.Resampling.LANCZOS
     elif hasattr(Image, 'LANCZOS'):
-         if not hasattr(Image, 'ANTIALIAS'): Image.ANTIALIAS = Image.LANCZOS
-    elif not hasattr(Image, 'ANTIALIAS'): print("WARNING: Pillow ANTIALIAS/Resampling issue.")
-except Exception as e_mp: print(f"WARNING: ANTIALIAS patch error: {e_mp}")
+        if not hasattr(Image, 'ANTIALIAS'): Image.ANTIALIAS = Image.LANCZOS
+    elif not hasattr(Image, 'ANTIALIAS'):
+        print("WARNING: Pillow version lacks common Resampling or ANTIALIAS. MoviePy effects might fail.")
+except Exception as e_mp:
+    print(f"WARNING: ANTIALIAS monkey-patch error: {e_mp}")
 
 logger = logging.getLogger(__name__)
-# logger.setLevel(logging.DEBUG)
+# logger.setLevel(logging.DEBUG) # Uncomment for very verbose debugging
 
 ELEVENLABS_CLIENT_IMPORTED = False; ElevenLabsAPIClient = None; Voice = None; VoiceSettings = None
 try:
@@ -68,216 +70,247 @@ class VisualEngine:
             except Exception as e_rwy_init: logger.error(f"Initial RunwayML client init failed: {e_rwy_init}"); self.USE_RUNWAYML = False
         logger.info("VisualEngine initialized.")
 
-    def set_openai_api_key(self, k): self.openai_api_key=k; self.USE_AI_IMAGE_GENERATION=bool(k); logger.info(f"DALL-E: {'Ready' if self.USE_AI_IMAGE_GENERATION else 'Disabled'}")
-    def set_elevenlabs_api_key(self, k, vid=None):
-        self.elevenlabs_api_key=k;
-        if vid: self.elevenlabs_voice_id = vid; logger.info(f"11L Voice ID updated to: {vid}")
-        if k and ELEVENLABS_CLIENT_IMPORTED and ElevenLabsAPIClient:
-            try: self.elevenlabs_client_instance = ElevenLabsAPIClient(api_key=k); self.USE_ELEVENLABS=True; logger.info(f"11L Client: Ready (Voice:{self.elevenlabs_voice_id})")
-            except Exception as e: logger.error(f"11L client init err: {e}. Disabled.", exc_info=True); self.USE_ELEVENLABS=False; self.elevenlabs_client_instance=None
+    def set_openai_api_key(self, api_key_value): self.openai_api_key = api_key_value; self.USE_AI_IMAGE_GENERATION = bool(api_key_value); logger.info(f"DALL-E status: {'Ready' if self.USE_AI_IMAGE_GENERATION else 'Disabled'}")
+    def set_elevenlabs_api_key(self, api_key_value, voice_id_from_secret=None):
+        self.elevenlabs_api_key = api_key_value
+        if voice_id_from_secret: self.elevenlabs_voice_id = voice_id_from_secret; logger.info(f"11L Voice ID updated to: {self.elevenlabs_voice_id} via set_elevenlabs_api_key.")
+        if api_key_value and ELEVENLABS_CLIENT_IMPORTED and ElevenLabsAPIClient:
+            try: self.elevenlabs_client_instance = ElevenLabsAPIClient(api_key=api_key_value); self.USE_ELEVENLABS = bool(self.elevenlabs_client_instance); logger.info(f"11L Client: {'Ready' if self.USE_ELEVENLABS else 'Failed'} (Voice: {self.elevenlabs_voice_id})")
+            except Exception as e_11l_setkey_init: logger.error(f"11L client init error: {e_11l_setkey_init}. Disabled.", exc_info=True); self.USE_ELEVENLABS=False; self.elevenlabs_client_instance=None
         else: self.USE_ELEVENLABS = False; logger.info(f"11L Disabled (key/SDK).")
-    def set_pexels_api_key(self, k): self.pexels_api_key=k; self.USE_PEXELS=bool(k); logger.info(f"Pexels: {'Ready' if self.USE_PEXELS else 'Disabled'}")
-    def set_runway_api_key(self, k):
-        self.runway_api_key = k
-        if k:
+    def set_pexels_api_key(self, api_key_value): self.pexels_api_key = api_key_value; self.USE_PEXELS = bool(api_key_value); logger.info(f"Pexels status: {'Ready' if self.USE_PEXELS else 'Disabled'}")
+    def set_runway_api_key(self, api_key_value):
+        self.runway_api_key = api_key_value
+        if api_key_value:
             if RUNWAYML_SDK_IMPORTED and RunwayMLAPIClientClass:
                 if not self.runway_ml_sdk_client_instance:
                     try:
-                        orig_secret = os.getenv("RUNWAYML_API_SECRET")
-                        if not orig_secret: os.environ["RUNWAYML_API_SECRET"]=k; logger.info("Temp set RUNWAYML_API_SECRET for SDK.")
-                        self.runway_ml_sdk_client_instance=RunwayMLAPIClientClass(); self.USE_RUNWAYML=True; logger.info("RunwayML Client init via set_key.")
-                        if not orig_secret: del os.environ["RUNWAYML_API_SECRET"]; logger.info("Cleared temp RUNWAYML_API_SECRET.")
-                    except Exception as e: logger.error(f"RunwayML Client init in set_key fail: {e}", exc_info=True); self.USE_RUNWAYML=False;self.runway_ml_sdk_client_instance=None
-                else: self.USE_RUNWAYML=True; logger.info("RunwayML Client already init.")
-            else: logger.warning("RunwayML SDK not imported. Disabled."); self.USE_RUNWAYML=False
-        else: self.USE_RUNWAYML=False; self.runway_ml_sdk_client_instance=None; logger.info("RunwayML Disabled (no key).")
+                        original_env_secret = os.getenv("RUNWAYML_API_SECRET")
+                        if not original_env_secret: os.environ["RUNWAYML_API_SECRET"] = api_key_value; logger.info("Temp set RUNWAYML_API_SECRET for SDK.")
+                        self.runway_ml_sdk_client_instance = RunwayMLAPIClientClass(); self.USE_RUNWAYML = True; logger.info("RunwayML Client init via set_runway_api_key.")
+                        if not original_env_secret: del os.environ["RUNWAYML_API_SECRET"]; logger.info("Cleared temp RUNWAYML_API_SECRET.")
+                    except Exception as e_runway_setkey_init: logger.error(f"RunwayML Client init in set_runway_api_key fail: {e_runway_setkey_init}", exc_info=True); self.USE_RUNWAYML=False;self.runway_ml_sdk_client_instance=None
+                else: self.USE_RUNWAYML = True; logger.info("RunwayML Client already init.")
+            else: logger.warning("RunwayML SDK not imported. Service disabled."); self.USE_RUNWAYML = False
+        else: self.USE_RUNWAYML = False; self.runway_ml_sdk_client_instance = None; logger.info("RunwayML Disabled (no API key).")
 
-    def _image_to_data_uri(self, img_path):
+    def _image_to_data_uri(self, image_path):
         try:
-            mime, _ = mimetypes.guess_type(img_path)
-            if not mime: ext=os.path.splitext(img_path)[1].lower(); mime_map={".png":"image/png",".jpg":"image/jpeg",".jpeg":"image/jpeg",".webp":"image/webp"}; mime=mime_map.get(ext,"application/octet-stream");
-            if mime=="application/octet-stream": logger.warning(f"Unknown MIME for {img_path}, using {mime}.")
-            with open(img_path,"rb") as f_img: enc_str=base64.b64encode(f_img.read()).decode('utf-8')
-            uri=f"data:{mime};base64,{enc_str}"; logger.debug(f"Data URI for {os.path.basename(img_path)} (MIME:{mime}): {uri[:100]}..."); return uri
-        except FileNotFoundError: logger.error(f"Img not found {img_path} for data URI."); return None
-        except Exception as e: logger.error(f"Error converting {img_path} to data URI:{e}",exc_info=True); return None
+            mime_type, _ = mimetypes.guess_type(image_path)
+            if not mime_type:
+                ext = os.path.splitext(image_path)[1].lower()
+                mime_map = {".png": "image/png", ".jpg": "image/jpeg", ".jpeg": "image/jpeg", ".webp": "image/webp"}
+                mime_type = mime_map.get(ext, "application/octet-stream")
+                if mime_type == "application/octet-stream": logger.warning(f"Could not determine MIME type for {image_path} from extension '{ext}', using default {mime_type}.")
+            with open(image_path, "rb") as image_file_handle: image_binary_data = image_file_handle.read()
+            encoded_base64_string = base64.b64encode(image_binary_data).decode('utf-8')
+            data_uri_string = f"data:{mime_type};base64,{encoded_base64_string}"
+            logger.debug(f"Generated data URI for {os.path.basename(image_path)} (MIME: {mime_type}). Data URI starts with: {data_uri_string[:100]}...")
+            return data_uri_string
+        except FileNotFoundError: logger.error(f"Image file not found at path: '{image_path}' when trying to create data URI."); return None
+        except Exception as e_data_uri_conversion: logger.error(f"Error converting image '{image_path}' to data URI: {e_data_uri_conversion}", exc_info=True); return None
 
-    def _map_resolution_to_runway_ratio(self, w, h):
-        r_str=f"{w}:{h}"; supp_r=["1280:720","720:1280","1104:832","832:1104","960:960","1584:672"];
-        if r_str in supp_r: return r_str
-        logger.warning(f"Res {r_str} not in Gen-4 list. Default 1280:720."); return "1280:720"
+    def _map_resolution_to_runway_ratio(self, width, height):
+        ratio_str=f"{width}:{height}";supported_ratios_gen4=["1280:720","720:1280","1104:832","832:1104","960:960","1584:672"];
+        if ratio_str in supported_ratios_gen4:return ratio_str
+        logger.warning(f"Res {ratio_str} not in Gen-4 list. Default 1280:720.");return "1280:720"
 
-    def _get_text_dimensions(self, txt, font):
-        dh=getattr(font,'size',self.active_font_size_pil);
-        if not txt: return 0,dh
+    def _get_text_dimensions(self, text_content, font_object_pil):
+        default_h = getattr(font_object_pil, 'size', self.active_font_size_pil)
+        if not text_content: return 0, default_h
         try:
-            if hasattr(font,'getbbox'):b=font.getbbox(txt);w=b[2]-b[0];h=b[3]-b[1];return w,h if h>0 else dh
-            elif hasattr(font,'getsize'):w,h=font.getsize(txt);return w,h if h>0 else dh
-            else: return int(len(txt)*dh*0.6),int(dh*1.2)
-        except Exception as e:logger.warning(f"Err _get_text_dimensions:{e}");return int(len(txt)*self.active_font_size_pil*0.6),int(self.active_font_size_pil*1.2)
+            if hasattr(font_object_pil,'getbbox'):bbox=font_object_pil.getbbox(text_content);w=bbox[2]-bbox[0];h=bbox[3]-bbox[1]; return w, h if h > 0 else default_h
+            elif hasattr(font_object_pil,'getsize'):w,h=font_object_pil.getsize(text_content); return w, h if h > 0 else default_h
+            else: return int(len(text_content)*default_h*0.6),int(default_h*1.2)
+        except Exception as e_getdim: logger.warning(f"Error in _get_text_dimensions: {e_getdim}"); return int(len(text_content)*self.active_font_size_pil*0.6),int(self.active_font_size_pil*1.2)
 
-    def _create_placeholder_image_content(self, desc, fname, sz=None):
-        # (Keep robust placeholder logic from before)
-        if sz is None: sz=self.video_frame_size; img=Image.new('RGB',sz,color=(20,20,40));drw=ImageDraw.Draw(img);pad=25;maxw=sz[0]-(2*pad);lns=[]
-        if not desc: desc="(Placeholder)"
-        wds=desc.split();curr_ln=""
-        for idx,w in enumerate(wds):
-            prosp_add=w+(" "if idx<len(wds)-1 else"");test_ln=curr_ln+prosp_add
-            curr_w,_=self._get_text_dimensions(test_ln,self.active_font_pil)
-            if curr_w==0 and test_ln.strip():curr_w=len(test_ln)*(self.active_font_size_pil*0.6)
-            if curr_w<=maxw:curr_ln=test_ln
+    def _create_placeholder_image_content(self,text_description,filename,size=None):
+        if size is None: size = self.video_frame_size
+        img = Image.new('RGB', size, color=(20, 20, 40)); d = ImageDraw.Draw(img); padding = 25
+        max_w = size[0] - (2 * padding); lines_for_placeholder = []
+        if not text_description: text_description = "(Placeholder Image)"
+        words_list = text_description.split(); current_line_buffer = ""
+        for word_idx, word_item in enumerate(words_list):
+            prospective_addition = word_item + (" " if word_idx < len(words_list) - 1 else "")
+            test_line_candidate = current_line_buffer + prospective_addition
+            current_w_text, _ = self._get_text_dimensions(test_line_candidate, self.active_font_pil)
+            if current_w_text == 0 and test_line_candidate.strip(): current_w_text = len(test_line_candidate) * (self.active_font_size_pil * 0.6)
+            if current_w_text <= max_w: current_line_buffer = test_line_candidate
             else:
-                if curr_ln.strip():lns.append(curr_ln.strip())
-                curr_ln=prosp_add
-        if curr_ln.strip():lns.append(curr_ln.strip())
-        if not lns and desc:
-            avg_cw,_=self._get_text_dimensions("W",self.active_font_pil);avg_cw=avg_cw or(self.active_font_size_pil*0.6)
-            cpl=int(maxw/avg_cw)if avg_cw>0 else 20;lns.append(desc[:cpl]+("..."if len(desc)>cpl else""))
-        elif not lns:lns.append("(PH Error)")
-        _,slh=self._get_text_dimensions("Ay",self.active_font_pil);slh=slh if slh>0 else self.active_font_size_pil+2
-        maxl=min(len(lns),(sz[1]-(2*pad))//(slh+2))if slh>0 else 1;maxl=max(1,maxl)
-        yp=pad+(sz[1]-(2*pad)-maxl*(slh+2))/2.0
-        for i in range(maxl):
-            lt=lns[i];lw,_=self._get_text_dimensions(lt,self.active_font_pil)
-            if lw==0 and lt.strip():lw=len(lt)*(self.active_font_size_pil*0.6)
-            xp=(sz[0]-lw)/2.0
-            try:drw.text((xp,yp),lt,font=self.active_font_pil,fill=(200,200,180))
-            except Exception as e:logger.error(f"Pillow d.text err:{e} for '{lt}'")
-            yp+=slh+2
-            if i==6 and maxl>7:
-                try:drw.text((xp,yp),"...",font=self.active_font_pil,fill=(200,200,180))
-                except Exception as e:logger.error(f"Pillow ellipsis err:{e}");break
-        fpath=os.path.join(self.output_dir,fname)
-        try:img.save(fpath);return fpath
-        except Exception as e:logger.error(f"Save PH img '{fpath}' err:{e}",exc_info=True);return None
+                if current_line_buffer.strip(): lines_for_placeholder.append(current_line_buffer.strip())
+                current_line_buffer = prospective_addition
+        if current_line_buffer.strip(): lines_for_placeholder.append(current_line_buffer.strip())
+        if not lines_for_placeholder and text_description:
+            avg_char_w_est, _ = self._get_text_dimensions("W", self.active_font_pil); avg_char_w_est = avg_char_w_est or (self.active_font_size_pil * 0.6)
+            chars_per_line_est = int(max_w / avg_char_w_est) if avg_char_w_est > 0 else 20
+            lines_for_placeholder.append(text_description[:chars_per_line_est] + ("..." if len(text_description) > chars_per_line_est else ""))
+        elif not lines_for_placeholder: lines_for_placeholder.append("(Placeholder Error)")
+        _, single_h = self._get_text_dimensions("Ay", self.active_font_pil); single_h = single_h if single_h > 0 else self.active_font_size_pil + 2
+        max_l = min(len(lines_for_placeholder), (size[1] - (2 * padding)) // (single_h + 2)) if single_h > 0 else 1; max_l = max(1, max_l)
+        y_p = padding + (size[1] - (2 * padding) - max_l * (single_h + 2)) / 2.0
+        for i_line in range(max_l):
+            line_txt_content = lines_for_placeholder[i_line]; line_w_val, _ = self._get_text_dimensions(line_txt_content, self.active_font_pil)
+            if line_w_val == 0 and line_txt_content.strip(): line_w_val = len(line_txt_content) * (self.active_font_size_pil * 0.6)
+            x_p = (size[0] - line_w_val) / 2.0
+            try: d.text((x_p, y_p), line_txt_content, font=self.active_font_pil, fill=(200, 200, 180))
+            except Exception as e_draw: logger.error(f"Pillow d.text error: {e_draw} for '{line_txt_content}'")
+            y_p += single_h + 2
+            if i_line == 6 and max_l > 7:
+                try: d.text((x_p, y_p), "...", font=self.active_font_pil, fill=(200, 200, 180))
+                except Exception as e_elip: logger.error(f"Pillow d.text ellipsis error: {e_elip}"); break
+        filepath_placeholder = os.path.join(self.output_dir, filename)
+        try: img.save(filepath_placeholder); return filepath_placeholder
+        except Exception as e_save: logger.error(f"Saving placeholder image '{filepath_placeholder}' error: {e_save}", exc_info=True); return None
 
-    def _search_pexels_image(self, q_str, out_fn_base):
-        # (Keep robust Pexels logic from before)
+    def _search_pexels_image(self, query_str, output_fn_base):
         if not self.USE_PEXELS or not self.pexels_api_key: return None
-        h={"Authorization":self.pexels_api_key};p={"query":q_str,"per_page":1,"orientation":"landscape","size":"large2x"}
-        base_n_px,_=os.path.splitext(out_fn_base);px_fn=base_n_px+f"_pexels_{random.randint(1000,9999)}.jpg";fp_px=os.path.join(self.output_dir,px_fn)
+        http_headers = {"Authorization": self.pexels_api_key}
+        http_params = {"query": query_str, "per_page": 1, "orientation": "landscape", "size": "large2x"}
+        base_name_px, _ = os.path.splitext(output_fn_base)
+        pexels_fn_str = base_name_px + f"_pexels_{random.randint(1000,9999)}.jpg"
+        file_path_px = os.path.join(self.output_dir, pexels_fn_str)
         try:
-            logger.info(f"Pexels: Search '{q_str}'");eff_q=" ".join(q_str.split()[:5]);p["query"]=eff_q
-            resp_px=requests.get("https://api.pexels.com/v1/search",headers=h,params=p,timeout=20);resp_px.raise_for_status();data_px=resp_px.json()
+            logger.info(f"Pexels: Searching for '{query_str}'")
+            eff_query_px = " ".join(query_str.split()[:5])
+            http_params["query"] = eff_query_px
+            response_px = requests.get("https://api.pexels.com/v1/search", headers=http_headers, params=http_params, timeout=20)
+            response_px.raise_for_status()
+            data_px = response_px.json()
             if data_px.get("photos") and len(data_px["photos"]) > 0:
-                ph_det=data_px["photos"][0];ph_url=ph_det.get("src",{}).get("large2x")
-                if not ph_url:logger.warning(f"Pexels: 'large2x' URL missing for '{eff_q}'.");return None
-                img_resp=requests.get(ph_url,timeout=60);img_resp.raise_for_status();img_pil=Image.open(io.BytesIO(img_resp.content))
-                if img_pil.mode!='RGB':img_pil=img_pil.convert('RGB')
-                img_pil.save(fp_px);logger.info(f"Pexels: Saved to {fp_px}");return fp_px
-            else:logger.info(f"Pexels: No photos for '{eff_q}'.");return None
-        except requests.exceptions.RequestException as e:logger.error(f"Pexels ReqExc '{q_str}':{e}",exc_info=False);return None
-        except Exception as e:logger.error(f"Pexels GenErr '{q_str}':{e}",exc_info=True);return None
+                photo_details_px = data_px["photos"][0]
+                photo_url_px = photo_details_px.get("src", {}).get("large2x")
+                if not photo_url_px: logger.warning(f"Pexels: 'large2x' URL missing for '{eff_query_px}'. Details: {photo_details_px}"); return None
+                image_response_px = requests.get(photo_url_px, timeout=60); image_response_px.raise_for_status()
+                img_pil_data_px = Image.open(io.BytesIO(image_response_px.content))
+                if img_pil_data_px.mode != 'RGB': img_pil_data_px = img_pil_data_px.convert('RGB')
+                img_pil_data_px.save(file_path_px); logger.info(f"Pexels: Image saved to {file_path_px}"); return file_path_px
+            else: logger.info(f"Pexels: No photos for '{eff_query_px}'."); return None
+        except requests.exceptions.RequestException as e_req_px: logger.error(f"Pexels: RequestException for '{query_str}': {e_req_px}", exc_info=False); return None
+        except Exception as e_px_gen: logger.error(f"Pexels: General error for '{query_str}': {e_px_gen}", exc_info=True); return None
 
-    def _generate_video_clip_with_runwayml(self, motion_prompt, input_img_path, scene_id_base_fn, duration_s=5):
-        # (Keep robust RunwayML placeholder/integration logic from before)
-        if not self.USE_RUNWAYML or not self.runway_ml_sdk_client_instance: logger.warning("RunwayML skip: Not enabled/client not init."); return None
-        if not input_img_path or not os.path.exists(input_img_path): logger.error(f"Runway Gen-4 needs input img. Invalid: {input_img_path}"); return None
-        img_data_uri = self._image_to_data_uri(input_img_path);
-        if not img_data_uri: return None
-        rwy_dur = 10 if duration_s >= 8 else 5; rwy_ratio = self._map_resolution_to_runway_ratio(self.video_frame_size[0],self.video_frame_size[1])
-        rwy_base_name,_=os.path.splitext(scene_id_base_fn);rwy_out_fn=rwy_base_name+f"_runway_gen4_d{rwy_dur}s.mp4";rwy_out_fp=os.path.join(self.output_dir,rwy_out_fn)
-        logger.info(f"Runway Gen-4 task: motion='{motion_prompt[:70]}...', img='{os.path.basename(input_img_path)}', dur={rwy_dur}s, ratio='{rwy_ratio}'")
+    def _generate_video_clip_with_runwayml(self, text_prompt_for_motion, input_image_path, scene_identifier_filename_base, target_duration_seconds=5):
+        if not self.USE_RUNWAYML or not self.runway_ml_sdk_client_instance: logger.warning("RunwayML not enabled/client not init. Skip video."); return None
+        if not input_image_path or not os.path.exists(input_image_path): logger.error(f"Runway Gen-4 needs input image. Path invalid: {input_image_path}"); return None
+        image_data_uri_str = self._image_to_data_uri(input_image_path)
+        if not image_data_uri_str: return None
+        runway_dur = 10 if target_duration_seconds >= 8 else 5
+        runway_ratio = self._map_resolution_to_runway_ratio(self.video_frame_size[0], self.video_frame_size[1])
+        base_name_for_runway_vid, _ = os.path.splitext(scene_identifier_filename_base); output_vid_fn = base_name_for_runway_vid + f"_runway_gen4_d{runway_dur}s.mp4"
+        output_vid_fp = os.path.join(self.output_dir, output_vid_fn)
+        logger.info(f"Runway Gen-4 task: motion='{text_prompt_for_motion[:100]}...', img='{os.path.basename(input_image_path)}', dur={runway_dur}s, ratio='{runway_ratio}'")
         try:
-            rwy_task_sub = self.runway_ml_sdk_client_instance.image_to_video.create(model='gen4_turbo',prompt_image=img_data_uri,prompt_text=motion_prompt,duration=rwy_dur,ratio=rwy_ratio)
-            rwy_task_id = rwy_task_sub.id; logger.info(f"Runway task ID: {rwy_task_id}. Polling...")
-            poll_s=10;max_p_count=36;poll_t_start=time.time()
-            while time.time()-poll_t_start < max_p_count*poll_s:
-                time.sleep(poll_s);rwy_task_det=self.runway_ml_sdk_client_instance.tasks.retrieve(id=rwy_task_id)
-                logger.info(f"Runway task {rwy_task_id} status: {rwy_task_det.status}")
-                if rwy_task_det.status=='SUCCEEDED':
-                    rwy_out_url=getattr(getattr(rwy_task_det,'output',None),'url',None) or (getattr(rwy_task_det,'artifacts',None)and rwy_task_det.artifacts and hasattr(rwy_task_det.artifacts[0],'url')and rwy_task_det.artifacts[0].url) or (getattr(rwy_task_det,'artifacts',None)and rwy_task_det.artifacts and hasattr(rwy_task_det.artifacts[0],'download_url')and rwy_task_det.artifacts[0].download_url)
-                    if not rwy_out_url:logger.error(f"Runway task {rwy_task_id} SUCCEEDED, no output URL. Details:{vars(rwy_task_det)if hasattr(rwy_task_det,'__dict__')else rwy_task_det}");return None
-                    logger.info(f"Runway task {rwy_task_id} SUCCEEDED. Downloading: {rwy_out_url}")
-                    vid_resp=requests.get(rwy_out_url,stream=True,timeout=300);vid_resp.raise_for_status()
-                    with open(rwy_out_fp,'wb')as f:
-                        for chk in vid_resp.iter_content(chunk_size=8192):f.write(chk)
-                    logger.info(f"Runway Gen-4 video saved: {rwy_out_fp}");return rwy_out_fp
-                elif rwy_task_det.status in['FAILED','ABORTED','ERROR']:
-                    rwy_err_msg=getattr(rwy_task_det,'error_message',None)or getattr(getattr(rwy_task_det,'output',None),'error',"Unknown Runway error.")
-                    logger.error(f"Runway task {rwy_task_id} status:{rwy_task_det.status}. Error:{rwy_err_msg}");return None
-            logger.warning(f"Runway task {rwy_task_id} timed out.");return None
-        except AttributeError as e:logger.error(f"RunwayML SDK AttrError:{e}. SDK methods changed?",exc_info=True);return None
-        except Exception as e:logger.error(f"Runway Gen-4 API error:{e}",exc_info=True);return None
+            task_submitted_runway = self.runway_ml_sdk_client_instance.image_to_video.create(model='gen4_turbo', prompt_image=image_data_uri_str, prompt_text=text_prompt_for_motion, duration=runway_dur, ratio=runway_ratio)
+            task_id_runway = task_submitted_runway.id; logger.info(f"Runway Gen-4 task ID: {task_id_runway}. Polling...")
+            poll_sec=10; max_poll_count=36; poll_start_time = time.time()
+            while time.time() - poll_start_time < max_poll_count * poll_sec:
+                time.sleep(poll_sec); task_details_runway = self.runway_ml_sdk_client_instance.tasks.retrieve(id=task_id_runway)
+                logger.info(f"Runway task {task_id_runway} status: {task_details_runway.status}")
+                if task_details_runway.status == 'SUCCEEDED':
+                    output_url_runway = getattr(getattr(task_details_runway,'output',None),'url',None) or \
+                                        (getattr(task_details_runway,'artifacts',None) and task_details_runway.artifacts and hasattr(task_details_runway.artifacts[0],'url')and task_details_runway.artifacts[0].url) or \
+                                        (getattr(task_details_runway,'artifacts',None) and task_details_runway.artifacts and hasattr(task_details_runway.artifacts[0],'download_url')and task_details_runway.artifacts[0].download_url)
+                    if not output_url_runway: logger.error(f"Runway task {task_id_runway} SUCCEEDED, but no output URL. Details: {vars(task_details_runway) if hasattr(task_details_runway,'__dict__') else task_details_runway}"); return None
+                    logger.info(f"Runway task {task_id_runway} SUCCEEDED. Downloading: {output_url_runway}")
+                    video_resp_get = requests.get(output_url_runway, stream=True, timeout=300); video_resp_get.raise_for_status()
+                    with open(output_vid_fp,'wb') as f_vid:
+                        for chunk_data in video_resp_get.iter_content(chunk_size=8192): f_vid.write(chunk_data)
+                    logger.info(f"Runway Gen-4 video saved: {output_vid_fp}"); return output_vid_fp
+                elif task_details_runway.status in ['FAILED','ABORTED','ERROR']:
+                    err_msg_runway = getattr(task_details_runway,'error_message',None) or getattr(getattr(task_details_runway,'output',None),'error',"Unknown Runway error.")
+                    logger.error(f"Runway task {task_id_runway} status: {task_details_runway.status}. Error: {err_msg_runway}"); return None
+            logger.warning(f"Runway task {task_id_runway} timed out."); return None
+        except AttributeError as ae_sdk: logger.error(f"RunwayML SDK AttrError: {ae_sdk}. SDK/methods changed?", exc_info=True); return None
+        except Exception as e_runway_gen: logger.error(f"Runway Gen-4 API error: {e_runway_gen}", exc_info=True); return None
 
-    def _create_placeholder_video_content(self, text_desc, fname, duration=4, size=None):
-        # (Keep robust placeholder video logic from before)
-        if size is None: size = self.video_frame_size; fp = os.path.join(self.output_dir, fname); tc = None
-        try: tc = TextClip(text_desc, fontsize=50, color='white', font=self.video_overlay_font, bg_color='black', size=size, method='caption').set_duration(duration); tc.write_videofile(fp, fps=24, codec='libx264', preset='ultrafast', logger=None, threads=2); logger.info(f"Generic placeholder video: {fp}"); return fp
-        except Exception as e: logger.error(f"Generic placeholder video error {fp}: {e}", exc_info=True); return None
+    def _create_placeholder_video_content(self, text_desc_ph, filename_ph, duration_ph=4, size_ph=None):
+        if size_ph is None: size_ph = self.video_frame_size
+        filepath_ph = os.path.join(self.output_dir, filename_ph)
+        text_clip_ph = None 
+        try:
+            text_clip_ph = TextClip(text_desc_ph, fontsize=50, color='white', font=self.video_overlay_font,
+                                bg_color='black', size=size_ph, method='caption').set_duration(duration_ph)
+            text_clip_ph.write_videofile(filepath_ph, fps=24, codec='libx264', preset='ultrafast', logger=None, threads=2)
+            logger.info(f"Generic placeholder video created: {filepath_ph}")
+            return filepath_ph
+        except Exception as e_ph_vid: 
+            logger.error(f"Failed to create generic placeholder video '{filepath_ph}': {e_ph_vid}", exc_info=True)
+            return None
         finally:
-            if tc and hasattr(tc, 'close'):
-                try: tc.close()
+            if text_clip_ph and hasattr(text_clip_ph, 'close'):
+                try: text_clip_ph.close()
                 except Exception as e_cl_phv: logger.warning(f"Ignoring error closing placeholder TextClip: {e_cl_phv}")
     
-    def generate_scene_asset(self, img_prompt, motion_prompt, scene_dict, scene_id_fn_base, gen_as_vid=False, rwy_dur=5):
-        # (Keep robust asset generation logic from before, ensuring parameters match)
-        asset_base_name,_=os.path.splitext(scene_id_fn_base); asset_info_obj={'path':None,'type':'none','error':True,'prompt_used':img_prompt,'error_message':'Asset gen init failed'}; base_img_path_for_rwy=None
-        base_img_fn = asset_base_name + ("_base_for_video.png" if gen_as_vid else ".png"); base_img_fp = os.path.join(self.output_dir, base_img_fn)
+    def generate_scene_asset(self, image_generation_prompt_text, motion_prompt_text_for_video,
+                             scene_data_dict, scene_identifier_fn_base,
+                             generate_as_video_clip_flag=False, runway_target_dur_val=5):
+        base_name_asset, _ = os.path.splitext(scene_identifier_fn_base)
+        asset_info_result = {'path': None, 'type': 'none', 'error': True, 'prompt_used': image_generation_prompt_text, 'error_message': 'Asset generation init failed'}
+        path_for_input_image_runway = None
+        fn_for_base_image = base_name_asset + ("_base_for_video.png" if generate_as_video_clip_flag else ".png")
+        fp_for_base_image = os.path.join(self.output_dir, fn_for_base_image)
         if self.USE_AI_IMAGE_GENERATION and self.openai_api_key:
-            max_r,att_c=2,0
-            for att_idx in range(max_r):
-                att_c=att_idx+1
-                try:
-                    logger.info(f"Att {att_c} DALL-E (base img): {img_prompt[:70]}...");oai_client=openai.OpenAI(api_key=self.openai_api_key,timeout=90.0);oai_resp=oai_client.images.generate(model=self.dalle_model,prompt=img_prompt,n=1,size=self.image_size_dalle3,quality="hd",response_format="url",style="vivid");oai_url=oai_resp.data[0].url;oai_rev_p=getattr(oai_resp.data[0],'revised_prompt',None)
-                    if oai_rev_p:logger.info(f"DALL-E revised: {oai_rev_p[:70]}...")
-                    oai_img_get_resp=requests.get(oai_url,timeout=120);oai_img_get_resp.raise_for_status();oai_pil_img=Image.open(io.BytesIO(oai_img_get_resp.content))
-                    if oai_pil_img.mode!='RGB':oai_pil_img=oai_pil_img.convert('RGB')
-                    oai_pil_img.save(base_img_fp);logger.info(f"DALL-E base img saved: {base_img_fp}");base_img_path_for_rwy=base_img_fp;asset_info_obj={'path':base_img_fp,'type':'image','error':False,'prompt_used':img_prompt,'revised_prompt':oai_rev_p};break
-                except openai.RateLimitError as e:logger.warning(f"OpenAI RateLimit Att {att_c}:{e}.Retry...");time.sleep(5*att_c);asset_info_obj['error_message']=str(e)
-                except openai.APIError as e:logger.error(f"OpenAI APIError Att {att_c}:{e}");asset_info_obj['error_message']=str(e);break
-                except requests.exceptions.RequestException as e:logger.error(f"Requests Err DALL-E Att {att_c}:{e}");asset_info_obj['error_message']=str(e);break
-                except Exception as e:logger.error(f"General DALL-E Err Att {att_c}:{e}",exc_info=True);asset_info_obj['error_message']=str(e);break
-            if asset_info_obj['error']:logger.warning(f"DALL-E failed after {att_c} attempts for base img.")
-        if asset_info_obj['error'] and self.USE_PEXELS:
-            logger.info("Trying Pexels for base img.");px_q=scene_dict.get('pexels_search_query_감독',f"{scene_dict.get('emotional_beat','')} {scene_dict.get('setting_description','')}");px_p=self._search_pexels_image(px_q,base_img_fn)
-            if px_p:base_img_path_for_rwy=px_p;asset_info_obj={'path':px_p,'type':'image','error':False,'prompt_used':f"Pexels:{px_q}"}
-            else:curr_err=asset_info_obj.get('error_message',"");asset_info_obj['error_message']=(curr_err+" Pexels failed for base.").strip()
-        if asset_info_obj['error']:
-            logger.warning("Base img (DALL-E/Pexels) failed. Using placeholder.");ph_p_txt=asset_info_obj.get('prompt_used',img_prompt);ph_img_p=self._create_placeholder_image_content(f"[Base Placeholder]{ph_p_txt[:70]}...",base_img_fn)
-            if ph_img_p:base_img_path_for_rwy=ph_img_p;asset_info_obj={'path':ph_img_p,'type':'image','error':False,'prompt_used':ph_p_txt}
-            else:curr_err=asset_info_obj.get('error_message',"");asset_info_obj['error_message']=(curr_err+" Base placeholder failed.").strip()
-        if gen_as_vid:
-            if not base_img_path_for_rwy:logger.error("RunwayML video: base img failed.");asset_info_obj['error']=True;asset_info_obj['error_message']=(asset_info_obj.get('error_message',"")+" Base img miss, Runway abort.").strip();asset_info_obj['type']='none';return asset_info_obj
+            max_r_dalle, attempt_count_dalle = 2,0;
+            for att_n_dalle in range(max_r_dalle):
+                attempt_count_dalle = att_n_dalle + 1
+                try: 
+                    logger.info(f"Att {attempt_count_dalle} DALL-E (base img): {image_generation_prompt_text[:70]}..."); oai_cl = openai.OpenAI(api_key=self.openai_api_key,timeout=90.0); oai_r = oai_cl.images.generate(model=self.dalle_model,prompt=image_generation_prompt_text,n=1,size=self.image_size_dalle3,quality="hd",response_format="url",style="vivid"); oai_iu = oai_r.data[0].url; oai_rp = getattr(oai_r.data[0],'revised_prompt',None);
+                    if oai_rp: logger.info(f"DALL-E revised: {oai_rp[:70]}...")
+                    oai_ir = requests.get(oai_iu,timeout=120); oai_ir.raise_for_status(); oai_id = Image.open(io.BytesIO(oai_ir.content));
+                    if oai_id.mode!='RGB': oai_id=oai_id.convert('RGB')
+                    oai_id.save(fp_for_base_image); logger.info(f"DALL-E base img saved: {fp_for_base_image}"); path_for_input_image_runway=fp_for_base_image; asset_info_result={'path':fp_for_base_image,'type':'image','error':False,'prompt_used':image_generation_prompt_text,'revised_prompt':oai_rp}; break 
+                except openai.RateLimitError as e_oai_rl: logger.warning(f"OpenAI RateLimit Att {attempt_count_dalle}:{e_oai_rl}.Retry...");time.sleep(5*attempt_count_dalle);asset_info_result['error_message']=str(e_oai_rl)
+                except openai.APIError as e_oai_api: logger.error(f"OpenAI APIError Att {attempt_count_dalle}:{e_oai_api}");asset_info_result['error_message']=str(e_oai_api);break
+                except requests.exceptions.RequestException as e_oai_req: logger.error(f"Requests Err DALL-E Att {attempt_count_dalle}:{e_oai_req}");asset_info_result['error_message']=str(e_oai_req);break
+                except Exception as e_oai_gen: logger.error(f"General DALL-E Err Att {attempt_count_dalle}:{e_oai_gen}",exc_info=True);asset_info_result['error_message']=str(e_oai_gen);break
+            if asset_info_result['error']: logger.warning(f"DALL-E failed after {attempt_count_dalle} attempts for base img.")
+        if asset_info_result['error'] and self.USE_PEXELS:
+            logger.info("Trying Pexels for base img.");px_qt=scene_data_dict.get('pexels_search_query_감독',f"{scene_data_dict.get('emotional_beat','')} {scene_data_dict.get('setting_description','')}");px_pp=self._search_pexels_image(px_qt,fn_for_base_image);
+            if px_pp:path_for_input_image_runway=px_pp;asset_info_result={'path':px_pp,'type':'image','error':False,'prompt_used':f"Pexels:{px_qt}"}
+            else:current_em_px=asset_info_result.get('error_message',"");asset_info_result['error_message']=(current_em_px+" Pexels failed for base.").strip()
+        if asset_info_result['error']:
+            logger.warning("Base img (DALL-E/Pexels) failed. Using placeholder.");ph_ppt=asset_info_result.get('prompt_used',image_generation_prompt_text);php=self._create_placeholder_image_content(f"[Base Placeholder]{ph_ppt[:70]}...",fn_for_base_image);
+            if php:path_for_input_image_runway=php;asset_info_result={'path':php,'type':'image','error':False,'prompt_used':ph_ppt}
+            else:current_em_ph=asset_info_result.get('error_message',"");asset_info_result['error_message']=(current_em_ph+" Base placeholder failed.").strip()
+        if generate_as_video_clip_flag:
+            if not path_for_input_image_runway:logger.error("RunwayML video: base img failed.");asset_info_result['error']=True;asset_info_result['error_message']=(asset_info_result.get('error_message',"")+" Base img miss, Runway abort.").strip();asset_info_result['type']='none';return asset_info_result
             if self.USE_RUNWAYML:
-                rwy_vid_p=self._generate_video_clip_with_runwayml(motion_prompt,base_img_path_for_rwy,asset_base_name,rwy_dur)
-                if rwy_vid_p and os.path.exists(rwy_vid_p):asset_info_obj={'path':rwy_vid_p,'type':'video','error':False,'prompt_used':motion_prompt,'base_image_path':base_img_path_for_rwy}
-                else:logger.warning(f"RunwayML video failed for {asset_base_name}. Fallback to base img.");asset_info_obj['error']=True;asset_info_obj['error_message']=(asset_info_obj.get('error_message',"Base img ok.")+" RunwayML video fail; use base img.").strip();asset_info_obj['path']=base_img_path_for_rwy;asset_info_obj['type']='image';asset_info_obj['prompt_used']=img_prompt
-            else:logger.warning("RunwayML selected but disabled. Use base img.");asset_info_obj['error']=True;asset_info_obj['error_message']=(asset_info_obj.get('error_message',"Base img ok.")+" RunwayML disabled; use base img.").strip();asset_info_obj['path']=base_img_path_for_rwy;asset_info_obj['type']='image';asset_info_obj['prompt_used']=img_prompt
-        return asset_info_obj
+                runway_video_p=self._generate_video_clip_with_runwayml(motion_prompt_text_for_video,path_for_input_image_runway,base_name_asset,runway_target_dur_val)
+                if runway_video_p and os.path.exists(runway_video_p):asset_info_result={'path':runway_video_p,'type':'video','error':False,'prompt_used':motion_prompt_text_for_video,'base_image_path':path_for_input_image_runway}
+                else:logger.warning(f"RunwayML video failed for {base_name_asset}. Fallback to base img.");asset_info_result['error']=True;asset_info_result['error_message']=(asset_info_result.get('error_message',"Base img ok.")+" RunwayML video fail; use base img.").strip();asset_info_result['path']=path_for_input_image_runway;asset_info_result['type']='image';asset_info_result['prompt_used']=image_generation_prompt_text
+            else:logger.warning("RunwayML selected but disabled. Use base img.");asset_info_result['error']=True;asset_info_result['error_message']=(asset_info_result.get('error_message',"Base img ok.")+" RunwayML disabled; use base img.").strip();asset_info_result['path']=path_for_input_image_runway;asset_info_result['type']='image';asset_info_result['prompt_used']=image_generation_prompt_text
+        return asset_info_result
 
-    def generate_narration_audio(self, narration_text, output_fn="narration_overall.mp3"):
-        # (Corrected version from previous response)
-        if not self.USE_ELEVENLABS or not self.elevenlabs_client_instance or not narration_text: logger.info("11L conditions not met. Skip audio."); return None
-        narration_fp = os.path.join(self.output_dir, output_fn)
+    def generate_narration_audio(self, text_to_narrate, output_filename="narration_overall.mp3"):
+        if not self.USE_ELEVENLABS or not self.elevenlabs_client_instance or not text_to_narrate: logger.info("11L conditions not met. Skip audio."); return None
+        audio_filepath_narration = os.path.join(self.output_dir, output_filename)
         try:
-            logger.info(f"11L audio (Voice:{self.elevenlabs_voice_id}): \"{narration_text[:70]}...\"")
-            stream_method = None
-            if hasattr(self.elevenlabs_client_instance,'text_to_speech') and hasattr(self.elevenlabs_client_instance.text_to_speech,'stream'): stream_method=self.elevenlabs_client_instance.text_to_speech.stream; logger.info("Using 11L .text_to_speech.stream()")
-            elif hasattr(self.elevenlabs_client_instance,'generate_stream'): stream_method=self.elevenlabs_client_instance.generate_stream; logger.info("Using 11L .generate_stream()")
-            elif hasattr(self.elevenlabs_client_instance,'generate'):
-                logger.info("Using 11L .generate() (non-streaming).")
-                voice_p = Voice(voice_id=str(self.elevenlabs_voice_id),settings=self.elevenlabs_voice_settings_obj) if Voice and self.elevenlabs_voice_settings_obj else str(self.elevenlabs_voice_id)
-                audio_b = self.elevenlabs_client_instance.generate(text=narration_text,voice=voice_p,model="eleven_multilingual_v2")
-                with open(narration_fp,"wb") as f_audio: f_audio.write(audio_b); logger.info(f"11L audio (non-stream): {narration_fp}"); return narration_fp
-            else: logger.error("No recognized 11L audio method."); return None
-            if stream_method:
-                voice_stream_params={"voice_id":str(self.elevenlabs_voice_id)}
+            logger.info(f"Generating 11L audio (Voice ID: {self.elevenlabs_voice_id}) for text: \"{text_to_narrate[:70]}...\"")
+            audio_stream_method_11l = None
+            if hasattr(self.elevenlabs_client_instance, 'text_to_speech') and hasattr(self.elevenlabs_client_instance.text_to_speech, 'stream'): audio_stream_method_11l = self.elevenlabs_client_instance.text_to_speech.stream; logger.info("Using 11L SDK: client.text_to_speech.stream()")
+            elif hasattr(self.elevenlabs_client_instance, 'generate_stream'): audio_stream_method_11l = self.elevenlabs_client_instance.generate_stream; logger.info("Using 11L SDK: client.generate_stream()")
+            elif hasattr(self.elevenlabs_client_instance, 'generate'):
+                logger.info("Using 11L SDK: client.generate() (non-streaming).")
+                voice_param_11l = str(self.elevenlabs_voice_id);
+                if Voice and self.elevenlabs_voice_settings_obj: voice_param_11l = Voice(voice_id=str(self.elevenlabs_voice_id), settings=self.elevenlabs_voice_settings_obj)
+                audio_bytes_data = self.elevenlabs_client_instance.generate(text=text_to_narrate, voice=voice_param_11l, model="eleven_multilingual_v2")
+                with open(audio_filepath_narration, "wb") as audio_file_out: audio_file_out.write(audio_bytes_data)
+                logger.info(f"11L audio (non-streamed) saved to: {audio_filepath_narration}"); return audio_filepath_narration
+            else: logger.error("No recognized audio generation method on 11L client."); return None
+            if audio_stream_method_11l:
+                params_for_voice_stream = {"voice_id": str(self.elevenlabs_voice_id)}
                 if self.elevenlabs_voice_settings_obj:
-                    if hasattr(self.elevenlabs_voice_settings_obj,'model_dump'): voice_stream_params["voice_settings"]=self.elevenlabs_voice_settings_obj.model_dump()
-                    elif hasattr(self.elevenlabs_voice_settings_obj,'dict'): voice_stream_params["voice_settings"]=self.elevenlabs_voice_settings_obj.dict()
-                    else: voice_stream_params["voice_settings"]=self.elevenlabs_voice_settings_obj
-                audio_iter = stream_method(text=narration_text,model_id="eleven_multilingual_v2",**voice_stream_params)
-                with open(narration_fp,"wb") as f_audio_stream:
-                    for chunk_item in audio_iter:
-                        if chunk_item: f_audio_stream.write(chunk_item)
-                logger.info(f"11L audio (stream): {narration_fp}"); return narration_fp
-        except AttributeError as e_11l_attr: logger.error(f"11L SDK AttrError: {e_11l_attr}. SDK/methods changed?", exc_info=True); return None
-        except Exception as e_11l_gen: logger.error(f"11L audio gen error: {e_11l_gen}", exc_info=True); return None
+                    if hasattr(self.elevenlabs_voice_settings_obj, 'model_dump'): params_for_voice_stream["voice_settings"] = self.elevenlabs_voice_settings_obj.model_dump()
+                    elif hasattr(self.elevenlabs_voice_settings_obj, 'dict'): params_for_voice_stream["voice_settings"] = self.elevenlabs_voice_settings_obj.dict()
+                    else: params_for_voice_stream["voice_settings"] = self.elevenlabs_voice_settings_obj
+                audio_data_iterator_11l = audio_stream_method_11l(text=text_to_narrate, model_id="eleven_multilingual_v2", **params_for_voice_stream)
+                with open(audio_filepath_narration, "wb") as audio_file_out_stream:
+                    for audio_chunk_data in audio_data_iterator_11l:
+                        if audio_chunk_data: audio_file_out_stream.write(audio_chunk_data)
+                logger.info(f"11L audio (streamed) saved to: {audio_filepath_narration}"); return audio_filepath_narration
+        except AttributeError as ae_11l_sdk: logger.error(f"AttributeError with 11L SDK client: {ae_11l_sdk}. SDK version/methods might differ.", exc_info=True); return None
+        except Exception as e_11l_general_audio: logger.error(f"General error during 11L audio generation: {e_11l_general_audio}", exc_info=True); return None
 
     def assemble_animatic_from_assets(self, asset_data_list, overall_narration_path=None, output_filename="final_video.mp4", fps=24):
-        # (Keep the version with robust image processing, C-contiguous array, debug saves, and pix_fmt)
-        # This is the most critical part for your "blank/corrupted video" issue.
         if not asset_data_list: logger.warning("No assets for animatic."); return None
         processed_moviepy_clips_list = []; narration_audio_clip_mvpy = None; final_video_output_clip = None
         logger.info(f"Assembling from {len(asset_data_list)} assets. Target Frame: {self.video_frame_size}.")
@@ -290,7 +323,7 @@ class VisualEngine:
             if not (path_of_asset and os.path.exists(path_of_asset)): logger.warning(f"S{num_of_scene}: Not found '{path_of_asset}'. Skip."); continue
             if duration_for_scene <= 0: logger.warning(f"S{num_of_scene}: Invalid duration ({duration_for_scene}s). Skip."); continue
             
-            active_scene_clip = None # Clip for this iteration
+            active_scene_clip = None
             try:
                 if type_of_asset == 'image':
                     pil_img_original = Image.open(path_of_asset)
@@ -314,28 +347,32 @@ class VisualEngine:
                     logger.debug(f"S{num_of_scene} (3-PasteOnRGBA): Image pasted onto transparent RGBA canvas. Mode:{canvas_for_compositing_rgba.mode}, Size:{canvas_for_compositing_rgba.size}")
                     canvas_for_compositing_rgba.save(os.path.join(self.output_dir,f"debug_3_COMPOSITED_RGBA_S{num_of_scene}.png"))
 
-                    final_rgb_image_for_pil = Image.new("RGB", self.video_frame_size, (0, 0, 0)) # Opaque black background
-                    if canvas_for_compositing_rgba.mode == 'RGBA':
-                        final_rgb_image_for_pil.paste(canvas_for_compositing_rgba, mask=canvas_for_compositing_rgba.split()[3])
+                    final_rgb_image_for_pil = Image.new("RGB", self.video_frame_size, (0, 0, 0))
+                    if canvas_for_compositing_rgba.mode == 'RGBA': final_rgb_image_for_pil.paste(canvas_for_compositing_rgba, mask=canvas_for_compositing_rgba.split()[3])
                     else: final_rgb_image_for_pil.paste(canvas_for_compositing_rgba)
                     logger.debug(f"S{num_of_scene} (4-ToRGB): Final RGB image created. Mode:{final_rgb_image_for_pil.mode}, Size:{final_rgb_image_for_pil.size}")
                     
                     debug_path_img_pre_numpy = os.path.join(self.output_dir,f"debug_4_PRE_NUMPY_RGB_S{num_of_scene}.png");
                     final_rgb_image_for_pil.save(debug_path_img_pre_numpy);
-                    logger.info(f"CRITICAL DEBUG: Saved PRE_NUMPY_RGB_S{num_of_scene} (image fed to NumPy) to {debug_path_img_pre_numpy}")
+                    logger.info(f"CRITICAL DEBUG: Saved PRE_NUMPY_RGB_S{num_of_scene} to {debug_path_img_pre_numpy}")
                     
                     numpy_frame_arr = np.array(final_rgb_image_for_pil, dtype=np.uint8)
                     if not numpy_frame_arr.flags['C_CONTIGUOUS']: numpy_frame_arr = np.ascontiguousarray(numpy_frame_arr, dtype=np.uint8)
-                    logger.debug(f"S{num_of_scene} (5-NumPy): Final NumPy array for MoviePy. Shape:{numpy_frame_arr.shape}, DType:{numpy_frame_arr.dtype}, Flags:{numpy_frame_arr.flags}")
-                    if numpy_frame_arr.size == 0 or numpy_frame_arr.ndim != 3 or numpy_frame_arr.shape[2] != 3: logger.error(f"S{num_of_scene}: Invalid NumPy array shape/size ({numpy_frame_arr.shape}) for ImageClip. Skipping."); continue
+                    logger.debug(f"S{num_of_scene} (5-NumPy): Final NumPy. Shape:{numpy_frame_arr.shape}, DType:{numpy_frame_arr.dtype}, Flags:{numpy_frame_arr.flags}")
+                    if numpy_frame_arr.size == 0 or numpy_frame_arr.ndim != 3 or numpy_frame_arr.shape[2] != 3: logger.error(f"S{num_of_scene}: Invalid NumPy shape/size ({numpy_frame_arr.shape}). Skipping."); continue
                     
                     base_image_clip_mvpy = ImageClip(numpy_frame_arr, transparent=False, ismask=False).set_duration(duration_for_scene)
                     logger.debug(f"S{num_of_scene} (6-ImageClip): Base ImageClip created. Duration: {base_image_clip_mvpy.duration}")
 
                     debug_path_moviepy_frame = os.path.join(self.output_dir,f"debug_7_MOVIEPY_FRAME_S{num_of_scene}.png")
-                    try: base_image_clip_mvpy.save_frame(debug_path_moviepy_frame, t=min(0.1, base_image_clip_mvpy.duration / 2 if base_image_clip_mvpy.duration > 0 else 0.1))
-                    logger.info(f"CRITICAL DEBUG: Saved frame FROM MOVIEPY ImageClip for S{num_of_scene} to {debug_path_moviepy_frame}")
-                    except Exception as e_save_mvpy_frame: logger.error(f"DEBUG: Error saving frame FROM MOVIEPY ImageClip S{num_of_scene}: {e_save_mvpy_frame}", exc_info=True)
+                    # <<< CORRECTED TRY-EXCEPT BLOCK FOR save_frame >>>
+                    try:
+                        save_frame_time = min(0.1, base_image_clip_mvpy.duration / 2 if base_image_clip_mvpy.duration > 0 else 0.1)
+                        base_image_clip_mvpy.save_frame(debug_path_moviepy_frame, t=save_frame_time) 
+                        logger.info(f"CRITICAL DEBUG: Saved frame FROM MOVIEPY ImageClip S{num_of_scene} to {debug_path_moviepy_frame}")
+                    except Exception as e_save_mvpy_frame:
+                        logger.error(f"DEBUG: Error saving frame FROM MOVIEPY ImageClip S{num_of_scene}: {e_save_mvpy_frame}", exc_info=True)
+                    # <<< END CORRECTION >>>
                     
                     fx_image_clip_mvpy = base_image_clip_mvpy
                     try:
@@ -345,7 +382,6 @@ class VisualEngine:
                     except Exception as e_kb_fx_loop: logger.error(f"S{num_of_scene} Ken Burns error: {e_kb_fx_loop}", exc_info=False)
                     active_scene_clip = fx_image_clip_mvpy
                 elif type_of_asset == 'video':
-                    # (Video processing logic as before)
                     source_video_clip_obj=None
                     try:
                         logger.debug(f"S{num_of_scene}: Loading VIDEO asset: {path_of_asset}")
@@ -365,7 +401,6 @@ class VisualEngine:
                             try: source_video_clip_obj.close()
                             except Exception as e_close_src_vid: logger.warning(f"S{num_of_scene}: Error closing source VideoFileClip: {e_close_src_vid}")
                 else: logger.warning(f"S{num_of_scene} Unknown asset type '{type_of_asset}'. Skipping."); continue
-                
                 if active_scene_clip and action_in_key:
                     try:
                         dur_text_overlay_val=min(active_scene_clip.duration-0.5,active_scene_clip.duration*0.8)if active_scene_clip.duration>0.5 else active_scene_clip.duration; start_text_overlay_val=0.25
@@ -381,7 +416,7 @@ class VisualEngine:
                 if active_scene_clip and hasattr(active_scene_clip,'close'):
                     try: active_scene_clip.close()
                     except Exception as e_close_active_err: logger.warning(f"S{num_of_scene}: Error closing active_scene_clip in error handler: {e_close_active_err}")
-                continue 
+                # Continue was removed to ensure loop proceeds, error is logged above. If continue is desired, it must be inside the try.
         if not processed_moviepy_clips_list: logger.warning("No MoviePy clips were successfully processed. Aborting animatic assembly before concatenation."); return None
         transition_duration_val=0.75
         try: