Update app.py

app.py

@@ -6,7 +6,7 @@ import logging
 # --- Streamlit PermissionError Mitigation Attempts ---
 if "STREAMLIT_CLIENT_GATHER_USAGE_STATS" not in os.environ:
     os.environ["STREAMLIT_CLIENT_GATHER_USAGE_STATS"] = "false"
-if "STREAMLIT_BROWSER_GATHERUSAGESTATS" not in os.environ:
+if "STREAMLIT_BROWSER_GATHERUSAGESTATS" not in os.environ: # For newer versions
     os.environ["STREAMLIT_BROWSER_GATHERUSAGESTATS"] = "false"
 streamlit_home_path_app = "/app/.streamlit_cai_config_v3" 
 if "STREAMLIT_HOME" not in os.environ and os.getcwd().startswith("/app"):
@@ -185,13 +185,13 @@ if not st.session_state.project_story_treatment_scenes_list: st.info("Use the si
 else:
     for i_main_display, scene_content_item_display in enumerate(st.session_state.project_story_treatment_scenes_list):
         scene_num_for_display = scene_content_item_display.get('scene_number', i_main_display + 1)
-        scene_title_for_display_main = scene_content_item_display.get('scene_title', 'Untitled Scene')
-        key_base_main_area_widgets = f"s{scene_num_for_display}_main_widgets_loop_v3_{i_main_display}" 
+        scene_title_for_display_main = scene_content_item_display.get('scene_title', 'Untitled Scene') # Used for caption
+        key_base_main_area_widgets = f"s{scene_num_for_display}_main_widgets_loop_v3_{i_main_display}"
 
         if "director_note" in scene_content_item_display and scene_content_item_display['director_note']: st.info(f"🎬 Director Note S{scene_num_for_display}: {scene_content_item_display['director_note']}")
         st.subheader(f"SCENE {scene_num_for_display}: {scene_title_for_display_main.upper()}"); 
         
-        treatment_display_col, visual_display_col = st.columns([0.45, 0.55]) # Defined here
+        treatment_display_col, visual_display_col = st.columns([0.45, 0.55]) 
 
         with treatment_display_col:
             with st.expander("📝 Scene Treatment & Controls", expanded=True):
@@ -217,21 +217,21 @@ else:
                 px_q_disp_main = scene_content_item_display.get('pexels_search_query_감독', None)
                 if px_q_disp_main: st.caption(f"Pexels Fallback: `{px_q_disp_main}`")
 
-        # <<< CORRECTED: Using visual_display_col as defined above >>>
-        with visual_display_col: 
+        with visual_display_col: # <<< USING THE CORRECT VARIABLE NAME HERE >>>
             asset_info_main_disp = st.session_state.project_generated_assets_info_list[i_main_display] if i_main_display < len(st.session_state.project_generated_assets_info_list) else None
             if asset_info_main_disp and not asset_info_main_disp.get('error') and asset_info_main_disp.get('path') and os.path.exists(asset_info_main_disp['path']):
                 path_asset_main = asset_info_main_disp['path']; type_asset_main = asset_info_main_disp.get('type','image')
-                if type_asset_main == 'image': st.image(path_asset_main, caption=f"S{scene_num_for_display} ({type_asset_main}): {scene_title_disp}")
+                # <<< CORRECTED VARIABLE NAME IN CAPTION F-STRING >>>
+                if type_asset_main == 'image': st.image(path_asset_main, caption=f"S{scene_num_for_display} ({type_asset_main}): {scene_title_for_display_main}")
                 elif type_asset_main == 'video':
                     try:
                         with open(path_asset_main,'rb') as vid_f_main: vid_b_main = vid_f_main.read()
-                        st.video(vid_b_main, format="video/mp4", start_time=0); st.caption(f"S{scene_num_for_display} ({type_asset_main}): {scene_title_disp}")
+                        st.video(vid_b_main, format="video/mp4", start_time=0); st.caption(f"S{scene_num_for_display} ({type_asset_main}): {scene_title_for_display_main}") # Corrected here too
                     except Exception as e_vid_disp_main_area: st.error(f"Error displaying video {path_asset_main}: {e_vid_disp_main_area}"); logger.error(f"Display video error: {e_vid_disp_main_area}", exc_info=True)
                 else: st.warning(f"Unknown asset type '{type_asset_main}' S{scene_num_for_display}.")
             else:
                 if st.session_state.project_story_treatment_scenes_list:
-                    err_msg_disp_main_area = asset_info_main_disp.get('error_message', 'Visual pending/failed.') if asset_info_main_disp else 'Visual pending/failed.'
+                    err_msg_disp_main_area = asset_info_main_disp.get('error_message', 'Visual pending or failed.') if asset_info_main_disp else 'Visual pending or failed.'
                     st.caption(err_msg_disp_main_area)
             
             with st.popover(f"✏️ Edit S{scene_num_for_display} Treatment"):
@@ -289,16 +289,16 @@ else:
     if st.session_state.project_story_treatment_scenes_list and any(asset_info_item_vid and not asset_info_item_vid.get('error') and asset_info_item_vid.get('path') for asset_info_item_vid in st.session_state.project_generated_assets_info_list if asset_info_item_vid is not None):
         if st.button("🎬 Assemble Narrated Cinematic Animatic", key="assemble_video_main_area_btn_final_unique_4", type="primary", use_container_width=True):
             with st.status("Assembling Ultra Animatic...", expanded=True) as status_video_assembly_final_op_main:
-                assets_for_final_vid_assembly_list = [] # Corrected initialization name
+                assets_for_final_vid_assembly_list = [] 
                 for i_vid_assembly_main_loop, scene_data_for_vid_assembly_main in enumerate(st.session_state.project_story_treatment_scenes_list):
                     asset_info_current_scene_for_vid_main = st.session_state.project_generated_assets_info_list[i_vid_assembly_main_loop] if i_vid_assembly_main_loop < len(st.session_state.project_generated_assets_info_list) else None
                     if asset_info_current_scene_for_vid_main and not asset_info_current_scene_for_vid_main.get('error') and asset_info_current_scene_for_vid_main.get('path') and os.path.exists(asset_info_current_scene_for_vid_main['path']):
-                        assets_for_final_vid_assembly_list.append({'path': asset_info_current_scene_for_vid_main['path'], 'type': asset_info_current_scene_for_vid_main.get('type', 'image'), 'scene_num': scene_data_for_vid_assembly_main.get('scene_number', i_vid_assembly_main_loop + 1), 'key_action': scene_data_for_vid_assembly_main.get('key_plot_beat', ''), 'duration': scene_data_for_vid_assembly_main.get('user_scene_duration_secs', DEFAULT_SCENE_DURATION_SECS)}) # Corrected append target
+                        assets_for_final_vid_assembly_list.append({'path': asset_info_current_scene_for_vid_main['path'], 'type': asset_info_current_scene_for_vid_main.get('type', 'image'), 'scene_num': scene_data_for_vid_assembly_main.get('scene_number', i_vid_assembly_main_loop + 1), 'key_action': scene_data_for_vid_assembly_main.get('key_plot_beat', ''), 'duration': scene_data_for_vid_assembly_main.get('user_scene_duration_secs', DEFAULT_SCENE_DURATION_SECS)})
                         status_video_assembly_final_op_main.write(f"Adding S{scene_data_for_vid_assembly_main.get('scene_number', i_vid_assembly_main_loop + 1)} ({asset_info_current_scene_for_vid_main.get('type')}).")
                     else: logger.warning(f"Skipping S{scene_data_for_vid_assembly_main.get('scene_number', i_vid_assembly_main_loop+1)} for video: No valid asset.")
-                if assets_for_final_vid_assembly_list: # Corrected check
+                if assets_for_final_vid_assembly_list:
                     status_video_assembly_final_op_main.write("Calling video engine..."); logger.info("APP: Calling visual_engine.assemble_animatic_from_assets")
-                    st.session_state.project_final_video_path = st.session_state.visual_content_engine.assemble_animatic_from_assets(asset_data_list=assets_for_final_vid_assembly_list, overall_narration_path=st.session_state.project_overall_narration_audio_path, output_filename="cinegen_ultra_animatic.mp4", fps=24) # Corrected pass
+                    st.session_state.project_final_video_path = st.session_state.visual_content_engine.assemble_animatic_from_assets(asset_data_list=assets_for_final_vid_assembly_list, overall_narration_path=st.session_state.project_overall_narration_audio_path, output_filename="cinegen_ultra_animatic.mp4", fps=24)
                     if st.session_state.project_final_video_path and os.path.exists(st.session_state.project_final_video_path): status_video_assembly_final_op_main.update(label="Ultra animatic assembled! 🎉", state="complete", expanded=False); st.balloons()
                     else: status_video_assembly_final_op_main.update(label="Video assembly failed. Check logs.", state="error", expanded=True); logger.error("APP: Video assembly returned None or file does not exist.")
                 else: status_video_assembly_final_op_main.update(label="No valid assets for video assembly.", state="error", expanded=True); logger.warning("APP: No valid assets found for video assembly.")