fix

app.py

@@ -147,48 +147,6 @@ def update_interface(responses, state):
             state
         )
 
-def skip_pair(state):
-    pair_index = state['pair_index']
-
-    # Update state
-    pair_index += 1
-    state['pair_index'] = pair_index
-
-    video1_url, video2_url = get_video_pair(state)
-    if video1_url is None:
-        # No more pairs
-        output_message = "Thank you for participating! No more videos."
-        return (
-            gr.update(visible=False),  # video_row
-            gr.update(value=""),       # video1
-            gr.update(value=""),       # video2
-            gr.update(visible=False),  # question_column
-            gr.update(visible=False),  # button_row
-            output_message,            # output
-            gr.update(value=None),     # q1
-            gr.update(value=None),     # q2
-            gr.update(value=None),     # q3
-            gr.update(interactive=False),  # next_btn
-            state
-        )
-    else:
-        video1_html = generate_video_html(video1_url)
-        video2_html = generate_video_html(video2_url)
-        # Update videos and reset questions
-        return (
-            gr.update(visible=True),   # video_row
-            gr.update(value=video1_html),  # video1
-            gr.update(value=video2_html),  # video2
-            gr.update(visible=True),   # question_column
-            gr.update(visible=True),   # button_row
-            "You chose to skip this pair.",  # output
-            gr.update(value=None),     # q1
-            gr.update(value=None),     # q2
-            gr.update(value=None),     # q3
-            gr.update(interactive=False),  # next_btn
-            state
-        )
-
 def check_all_answers(visual_quality, temporal_consistency, realistic_dynamics):
     if visual_quality and temporal_consistency and realistic_dynamics:
         return gr.update(interactive=True)
@@ -204,6 +162,7 @@ def authenticate_user(email, state):
     if not email or not is_valid_email(email):
         return (
             gr.update(),            # email_input (remains visible)
+            gr.update(),            # submit_email (remains visible)
             "Please enter a valid email.",  # email_output
             gr.update(visible=False),  # video_row
             gr.update(value=""),       # video1
@@ -238,7 +197,8 @@ def authenticate_user(email, state):
         if video1_url is None:
             output_message = "No new videos available for you."
             return (
-                gr.update(),            # email_input
+                gr.update(visible=False),  # email_input
+                gr.update(visible=False),  # submit_email
                 "",                     # email_output
                 gr.update(visible=False),  # video_row
                 gr.update(value=""),       # video1
@@ -253,6 +213,7 @@ def authenticate_user(email, state):
             video2_html = generate_video_html(video2_url)
             return (
                 gr.update(visible=False),  # email_input
+                gr.update(visible=False),  # submit_email
                 "",                        # email_output
                 gr.update(visible=True),   # video_row
                 gr.update(value=video1_html),  # video1
@@ -302,7 +263,7 @@ with gr.Blocks() as demo:
     # Buttons (initially hidden)
     with gr.Row(visible=False) as button_row:
         next_btn = gr.Button("Next", interactive=False)
-        skip_btn = gr.Button("Skip")
+        # skip_btn = gr.Button("Skip")  # Commented out if not used
 
     output = gr.Markdown()
 
@@ -330,22 +291,6 @@ with gr.Blocks() as demo:
         ]
     )
 
-    skip_btn.click(
-        fn=skip_pair,
-        inputs=state,
-        outputs=[
-            video_row,        # Update video_row
-            video1,           # Update video1
-            video2,           # Update video2
-            question_column,  # Update question_column
-            button_row,       # Update button_row
-            output,           # Update output message
-            q1, q2, q3,       # Reset questions
-            next_btn,         # Update next_btn
-            state
-        ]
-    )
-
     def on_change(visual_quality, temporal_consistency, realistic_dynamics):
         return check_all_answers(visual_quality, temporal_consistency, realistic_dynamics)
 
@@ -369,14 +314,15 @@ with gr.Blocks() as demo:
         fn=authenticate_user,
         inputs=[email_input, state],
         outputs=[
-            email_input,      # Hide email_input after submission
-            email_output,     # Clear email_output
-            video_row,        # Show video_row
+            email_input,      # Update email_input
+            submit_email,     # Update submit_email
+            email_output,     # Update email_output
+            video_row,        # Update video_row
             video1,           # Update video1
             video2,           # Update video2
-            question_column,  # Show question_column
-            button_row,       # Show button_row
-            output,           # Clear output message
+            question_column,  # Update question_column
+            button_row,       # Update button_row
+            output,           # Update output message
             state
         ]
     )