Update src/exp3-pilot.py

src/exp3-pilot.py

@@ -31,7 +31,7 @@ def get_google_creds():
     service_account_json = os.getenv("SERVICE_ACCOUNT_JSON")
     if service_account_json:
         creds_dict = json.loads(service_account_json)
-        scope = ["https://sheets.google.com/feeds", "https://www.googleapis.com/auth/drive"]
+        scope = ["https://spreadsheets.google.com/feeds", "https://www.googleapis.com/auth/drive"]
         creds = ServiceAccountCredentials.from_json_keyfile_dict(creds_dict, scope)
         return gspread.authorize(creds)
     else:
@@ -264,15 +264,15 @@ def render_initial_login_page():
             worker_id = int(participant_input)
             st.session_state.worker_id = worker_id
             st.session_state.step = "welcome"
-            st.rerun() # Trigger a rerun to go to the welcome page
+            st.rerun() # Immediately rerun to transition to the welcome page
         except ValueError:
             st.error("Please enter a valid numeric ID.")
-    st.stop() # Stop further execution for this run
+    st.stop() # Ensure nothing else renders after this page
 
 
 def render_welcome_page():
     """Renders the welcome and first instruction page."""
-    apply_css()
+    apply_css() # Apply general CSS
     st.markdown("<h1 class='compact-main-title'>Welcome to the Experiment 3!</h1>", unsafe_allow_html=True)
     st.markdown("<p class='instruction-text'>This is your final experiment, at last!</p>", unsafe_allow_html=True)
     st.markdown("<p class='instruction-text'>Please read the following instructions carefully and let us know if you have any questions.</p>", unsafe_allow_html=True)
@@ -280,9 +280,9 @@ def render_welcome_page():
     st.markdown("<div class='next-button-container'>", unsafe_allow_html=True)
     if st.button("Next", key="welcome_next_btn"):
         st.session_state.step = "instructions_1"
-        st.rerun() # Trigger a rerun to go to the next instruction page
+        st.rerun()
     st.markdown("</div>", unsafe_allow_html=True)
-    st.stop() # Stop further execution for this run
+    st.stop()
 
 
 def render_instructions_page_1():
@@ -301,13 +301,13 @@ def render_instructions_page_1():
         st.markdown("<div class='prev-button-container'>", unsafe_allow_html=True)
         if st.button("Previous", key="prev_1_btn"):
             st.session_state.step = "welcome"
-            st.rerun() # Trigger a rerun to go to the previous page
+            st.rerun()
         st.markdown("</div>", unsafe_allow_html=True)
     with col_next:
         st.markdown("<div class='next-button-container'>", unsafe_allow_html=True)
         if st.button("Next", key="next_1_btn"):
             st.session_state.step = "instructions_2"
-            st.rerun() # Trigger a rerun to go to the next page
+            st.rerun()
         st.markdown("</div>", unsafe_allow_html=True)
     st.stop()
 
@@ -413,10 +413,10 @@ def render_instructions_page_4():
         st.markdown("<div class='next-button-container'>", unsafe_allow_html=True)
         if st.button("Start Experiment", key="start_experiment_btn_final"):
             st.session_state.step = "training"
-            st.session_state.current_sample_index = 0 # Initialize sample index for the experiment
-            st.session_state[f'start_time_{st.session_state.current_sample_index}'] = time.time() # Start timer for first sample
+            st.session_state.current_sample_index = 0
+            st.session_state[f'start_time_{st.session_state.current_sample_index}'] = time.time()
             st.rerun()
-        st.markdown("</div>", unsafe_allow_html=True)
+    st.markdown("</div>", unsafe_allow_html=True)
     st.stop()
 
 
@@ -427,14 +427,11 @@ def render_main_experiment_ui():
 
     current_sample_index = st.session_state.get('current_sample_index', 0)
 
-    # If all samples are completed, transition to the training complete page
     if current_sample_index >= len(SAMPLES):
         st.session_state.step = "training_complete"
         st.rerun()
         st.stop() # Immediately stop after setting state and rerunning
 
-    # Record the start time when the sample is first loaded/displayed
-    # This ensures the timer starts when a *new* sample appears.
     if f'start_time_{current_sample_index}' not in st.session_state:
         st.session_state[f'start_time_{current_sample_index}'] = time.time()
 
@@ -562,7 +559,6 @@ def render_main_experiment_ui():
             
         st.markdown("---")
 
-        # Validation logic and button actions
         def validate_and_proceed():
             all_questions_answered = True
             part1_radio_key = f"part1_overall_evaluation_bottom_tab_{current_sample_index}"
@@ -589,7 +585,6 @@ def render_main_experiment_ui():
                 return False
             return True
 
-        # Next/Complete buttons
         if current_sample_index < len(SAMPLES) - 1:
             if st.button("Next Sample"):
                 if validate_and_proceed():
@@ -607,9 +602,7 @@ def render_main_experiment_ui():
                     st.session_state.responses.append(current_responses)
 
                     st.session_state.current_sample_index += 1
-                    # Set start time for the NEW sample
-                    st.session_state[f'start_time_{st.session_state.current_sample_index}'] = time.time()
-                    st.rerun() # Rerun to display the next sample
+                    st.rerun()
         else:
             if st.button("Complete Experiment"):
                 if validate_and_proceed():
@@ -627,7 +620,7 @@ def render_main_experiment_ui():
                     st.session_state.responses.append(current_responses)
 
                     st.session_state.step = "training_complete"
-                    st.rerun() # Rerun to display the completion page
+                    st.rerun()
     st.stop()
 
 
@@ -669,16 +662,17 @@ if __name__ == "__main__":
         st.session_state.responses = []
 
     # --- Set page config ONCE at the very top ---
-    # This remains the recommended best practice, even if dynamic.
-    # Streamlit will re-apply it on each rerun. The warning is usually harmless.
+    # This is critical for avoiding warnings and ensuring consistent layout behavior.
+    # Choose a default layout that works for most pages, or decide dynamically for the first render.
+    # Given your app shifts between centered and wide, we'll keep the dynamic set_page_config
+    # but acknowledge it might produce a warning. The key is that it's *before* other st calls.
     if st.session_state.step == "training":
         st.set_page_config(layout="wide")
     else:
         st.set_page_config(layout="centered")
 
     # Conditional rendering based on `st.session_state.step`
-    # Each branch must call its specific page rendering function,
-    # and that function MUST end with st.stop()
+    # Each branch must end with st.stop() to prevent fall-through
     if st.session_state.step == "initial_login":
         render_initial_login_page()
     elif st.session_state.step == "welcome":