Update app.py

app.py

@@ -758,19 +758,27 @@ with gr.Blocks() as demo:
                 height = gr.Slider(label="Height", minimum=256, maximum=2048, step=8, value=1024)
                 width = gr.Slider(label="Width", minimum=256, maximum=2048, step=8, value=1024)
 
+    # State to prevent circular updates between 3D control and sliders
+    is_syncing_from_3d = gr.State(False)
+    
     # --- Helper Functions ---
     def update_prompt_from_sliders(rotate, forward, tilt, wide):
         return build_camera_prompt(rotate, forward, tilt, wide)
     
     def sync_3d_to_sliders(camera_value):
+        """Sync 3D control values to sliders. Returns syncing flag = True."""
         if camera_value and isinstance(camera_value, dict):
             rot = camera_value.get('rotate_deg', 0)
             fwd = camera_value.get('move_forward', 0)
             tilt = camera_value.get('vertical_tilt', 0)
             wide = camera_value.get('wideangle', False)
             prompt = build_camera_prompt(rot, fwd, tilt, wide)
-            return rot, fwd, tilt, wide, prompt
-        return gr.update(), gr.update(), gr.update(), gr.update(), gr.update()
+            return rot, fwd, tilt, wide, prompt, True  # Set syncing flag
+        return gr.update(), gr.update(), gr.update(), gr.update(), gr.update(), False
+    
+    def clear_syncing_flag():
+        """Clear the syncing flag after sync is complete."""
+        return False
     
     def sync_sliders_to_3d(rotate, forward, tilt, wide):
         return {"rotate_deg": rotate, "move_forward": forward, "vertical_tilt": tilt, "wideangle": wide}
@@ -784,6 +792,20 @@ with gr.Blocks() as demo:
         data_url = f"data:image/png;base64,{img_str}"
         return gr.update(imageUrl=data_url)
     
+    def sync_3d_on_slider_change_if_not_syncing(is_syncing, img, rot, fwd, tilt, wide):
+        """Only sync sliders to 3D if we're not already syncing from 3D."""
+        if is_syncing:
+            # Skip update to prevent circular trigger
+            return gr.update()
+        camera_value = {"rotate_deg": rot, "move_forward": fwd, "vertical_tilt": tilt, "wideangle": wide}
+        if img is not None:
+            buffered = BytesIO()
+            img.save(buffered, format="PNG")
+            img_str = base64.b64encode(buffered.getvalue()).decode()
+            data_url = f"data:image/png;base64,{img_str}"
+            return gr.update(value=camera_value, imageUrl=data_url)
+        return gr.update(value=camera_value)
+    
     # Define inputs/outputs
     inputs = [image, rotate_deg, move_forward, vertical_tilt, wideangle, seed, randomize_seed, true_guidance_scale, num_inference_steps, height, width, prev_output]
     outputs = [result, seed, prompt_preview]
@@ -804,18 +826,22 @@ with gr.Blocks() as demo:
         slider.change(fn=update_prompt_from_sliders, inputs=[rotate_deg, move_forward, vertical_tilt, wideangle], outputs=[prompt_preview])
     wideangle.change(fn=update_prompt_from_sliders, inputs=[rotate_deg, move_forward, vertical_tilt, wideangle], outputs=[prompt_preview])
     
-    # 3D control -> Sliders + Prompt + Inference
+    # 3D control -> Sliders + Prompt + Inference (with syncing flag)
     camera_3d.change(
         fn=sync_3d_to_sliders,
         inputs=[camera_3d],
-        outputs=[rotate_deg, move_forward, vertical_tilt, wideangle, prompt_preview]
+        outputs=[rotate_deg, move_forward, vertical_tilt, wideangle, prompt_preview, is_syncing_from_3d]
     ).then(
         fn=maybe_infer,
         inputs=control_inputs_with_flag,
         outputs=outputs + [create_video_button]
+    ).then(
+        fn=clear_syncing_flag,
+        inputs=None,
+        outputs=[is_syncing_from_3d]
     )
     
-    # Sliders -> 3D control
+    # Sliders -> 3D control (only on user release, not programmatic changes)
     for slider in [rotate_deg, move_forward, vertical_tilt]:
         slider.release(fn=sync_sliders_to_3d, inputs=[rotate_deg, move_forward, vertical_tilt, wideangle], outputs=[camera_3d])
     wideangle.input(fn=sync_sliders_to_3d, inputs=[rotate_deg, move_forward, vertical_tilt, wideangle], outputs=[camera_3d])
@@ -845,7 +871,7 @@ with gr.Blocks() as demo:
     
     image.clear(fn=lambda: gr.update(imageUrl=None), outputs=[camera_3d])
     
-    # Live updates on slider release
+    # Live updates on slider release (user interaction only)
     for control in [rotate_deg, move_forward, vertical_tilt]:
         control.release(fn=maybe_infer, inputs=control_inputs_with_flag, outputs=outputs + [create_video_button])
     wideangle.input(fn=maybe_infer, inputs=control_inputs_with_flag, outputs=outputs + [create_video_button])
@@ -870,19 +896,13 @@ with gr.Blocks() as demo:
     )
     
     # Sync 3D component when sliders change (covers example loading)
-    def sync_3d_on_slider_change(img, rot, fwd, tilt, wide):
-        camera_value = {"rotate_deg": rot, "move_forward": fwd, "vertical_tilt": tilt, "wideangle": wide}
-        if img is not None:
-            buffered = BytesIO()
-            img.save(buffered, format="PNG")
-            img_str = base64.b64encode(buffered.getvalue()).decode()
-            data_url = f"data:image/png;base64,{img_str}"
-            return gr.update(value=camera_value, imageUrl=data_url)
-        return gr.update(value=camera_value)
-    
-    # When any slider value changes (including from examples), sync the 3D component
+    # Uses syncing flag to prevent circular updates
     for slider in [rotate_deg, move_forward, vertical_tilt]:
-        slider.change(fn=sync_3d_on_slider_change, inputs=[image, rotate_deg, move_forward, vertical_tilt, wideangle], outputs=[camera_3d])
+        slider.change(
+            fn=sync_3d_on_slider_change_if_not_syncing, 
+            inputs=[is_syncing_from_3d, image, rotate_deg, move_forward, vertical_tilt, wideangle], 
+            outputs=[camera_3d]
+        )
     
     gr.api(infer_camera_edit, api_name="infer_edit_camera_angles")
     gr.api(create_video_between_images, api_name="create_video_between_images")