Update app.py

app.py

@@ -18,7 +18,6 @@ st.set_page_config(page_title="Interdependence Concept Game", page_icon="🧬",
 # ==============================
 SCENARIOS = {
     "Blood Pressure (Hypotension)": [
-        # Loop 1: Baroreflex (nervous)
         {
             "loop_name": "Nervous loop (Baroreflex)",
             "sensor_options":  ["Chemoreceptor", "Mechanoreceptor", "Thermoreceptor"],
@@ -27,19 +26,14 @@ SCENARIOS = {
             "control_correct": "Medulla",
             "effector_options":["Pancreas", "Skeletal Muscle", "Heart and Blood Vessels"],
             "effector_correct":"Heart and Blood Vessels",
-            # Stage 2: Sensor -> Control
             "stage2_desc": "Afferent neurons release neurotransmitters (hydrophilic) across a synapse to the control center neuron.",
             "stage2_sig":  "Paracrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Concentration",
-            "stage2_feedback_wrong": "For synapses, diffusion across a tiny gap is primary (concentration gradient).",
-            # Stage 3: Control -> Effector
             "stage3_desc": "Controller neurons signal effectors via neurotransmitters (hydrophilic) across synapses.",
             "stage3_sig":  "Paracrine",
             "stage3_rec":  "Cell membrane",
             "stage3_grad": "Concentration",
-            "stage3_feedback_wrong": "Same synaptic logic: diffusion across a small gap.",
-            # Stage 4: Outcome
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Cardiac output increases and vasoconstriction increases",
@@ -48,7 +42,6 @@ SCENARIOS = {
             ],
             "outcome_correct": "Cardiac output increases and vasoconstriction increases",
         },
-        # Loop 2: RAAS (renal)
         {
             "loop_name": "Renal loop (RAAS)",
             "sensor_options":  ["Juxtaglomerular apparatus (kidney)", "Baroreceptors (carotid sinus)", "Osmoreceptors (hypothalamus)"],
@@ -61,12 +54,10 @@ SCENARIOS = {
             "stage2_sig":  "Endocrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Pressure",
-            "stage2_feedback_wrong": "Primary long-distance transport in blood is pressure-driven bulk flow; diffusion matters locally.",
             "stage3_desc": "Aldosterone (lipophilic steroid) travels **in the blood circulation** to tubular cells.",
             "stage3_sig":  "Endocrine",
             "stage3_rec":  "Inside the cell",
             "stage3_grad": "Pressure",
-            "stage3_feedback_wrong": "Again, long-distance delivery in blood is pressure-driven; diffusion is local.",
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Sodium & water reabsorption increases",
@@ -78,7 +69,6 @@ SCENARIOS = {
     ],
 
     "Glucose (Post-meal Hyperglycemia)": [
-        # Loop 1: Insulin
         {
             "loop_name": "Insulin loop",
             "sensor_options":  ["Pancreatic beta cells", "Pancreatic alpha cells", "Chemoreceptors (carotid)"],
@@ -91,12 +81,10 @@ SCENARIOS = {
             "stage2_sig":  "Endocrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Pressure",
-            "stage2_feedback_wrong":"For long-distance delivery in blood, the primary transport is pressure-driven bulk flow.",
             "stage3_desc": "The messenger reaches distant tissues via circulation to increase glucose uptake.",
             "stage3_sig":  "Endocrine",
             "stage3_rec":  "Cell membrane",
             "stage3_grad": "Pressure",
-            "stage3_feedback_wrong":"Primary transport in blood is pressure-driven.",
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Cellular glucose uptake increases",
@@ -105,7 +93,6 @@ SCENARIOS = {
             ],
             "outcome_correct": "Cellular glucose uptake increases",
         },
-        # Loop 2: Kidney overflow/excretion
         {
             "loop_name": "Kidney excretion loop (overflow)",
             "sensor_options":  ["Kidney (proximal tubule)", "Pancreatic beta cells", "Baroreceptors (carotid)"],
@@ -122,12 +109,10 @@ SCENARIOS = {
             "stage2_sig":  "Paracrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Concentration",
-            "stage2_feedback_wrong":"Local paracrine movement is diffusion-dominant (concentration gradient).",
             "stage3_desc": "Local control continues at transporters within the same tissue.",
             "stage3_sig":  "Paracrine",
             "stage3_rec":  "Cell membrane",
             "stage3_grad": "Concentration",
-            "stage3_feedback_wrong":"Local paracrine movement is diffusion-dominant.",
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Glucose excretion into the urine increases",
@@ -139,7 +124,6 @@ SCENARIOS = {
     ],
 
     "Temperature (Hypothermia)": [
-        # Loop 1: Shivering (somatic motor)
         {
             "loop_name": "Shivering loop (somatic motor)",
             "sensor_options":  ["Thermoreceptor", "Mechanoreceptor", "Chemoreceptor"],
@@ -152,12 +136,10 @@ SCENARIOS = {
             "stage2_sig":  "Paracrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Concentration",
-            "stage2_feedback_wrong":"Synaptic transmission uses diffusion across a tiny space (concentration gradient).",
             "stage3_desc": "Motor pathways direct skeletal muscle via neurotransmitters across synapses.",
             "stage3_sig":  "Paracrine",
             "stage3_rec":  "Cell membrane",
             "stage3_grad": "Concentration",
-            "stage3_feedback_wrong":"Synaptic diffusion across a small gap.",
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Muscle contraction",
@@ -166,7 +148,6 @@ SCENARIOS = {
             ],
             "outcome_correct": "Muscle contraction",
         },
-        # Loop 2: Skin vessels
         {
             "loop_name": "Skin vessel loop (vasoconstriction/vasodilation)",
             "sensor_options":  ["Thermoreceptor", "Mechanoreceptor", "Chemoreceptor"],
@@ -179,12 +160,10 @@ SCENARIOS = {
             "stage2_sig":  "Paracrine",
             "stage2_rec":  "Cell membrane",
             "stage2_grad": "Concentration",
-            "stage2_feedback_wrong":"Synaptic diffusion dominates at short range.",
             "stage3_desc": "Autonomic outputs alter peripheral vessel tone via neurotransmitters (synaptic).",
             "stage3_sig":  "Paracrine",
             "stage3_rec":  "Cell membrane",
             "stage3_grad":  "Concentration",
-            "stage3_feedback_wrong":"Local neurotransmitter diffusion across synapses.",
             "outcome_question": "What will be the following response?",
             "outcome_options": [
                 "Cutaneous vasoconstriction increases",
@@ -207,15 +186,15 @@ def init_state():
     if "assign" not in st.session_state:
         st.session_state.assign = {"sensor": None, "control": None, "effector": None}
     if "stage" not in st.session_state:
-        st.session_state.stage = 1     # 1..4
+        st.session_state.stage = 1
     if "stage_token" not in st.session_state:
-        st.session_state.stage_token = None  # set when Stage 1 is passed
+        st.session_state.stage_token = None
     if "msgs" not in st.session_state:
         st.session_state.msgs = {"s1":"", "s2":"", "s3":"", "s4":""}
     if "progress" not in st.session_state:
         st.session_state.progress = {sc: [False]*len(SCENARIOS[sc]) for sc in SCENARIOS}
     if "nonce" not in st.session_state:
-        st.session_state.nonce = 0     # forces unique widget keys per loop
+        st.session_state.nonce = 0
 
 def current_loop():
     return SCENARIOS[st.session_state.scenario][st.session_state.loop_idx]
@@ -237,7 +216,7 @@ def reset_loop():
     st.session_state.stage = 1
     st.session_state.stage_token = None
     st.session_state.msgs = {"s1":"", "s2":"", "s3":"", "s4":""}
-    st.session_state.nonce += 1  # ensures every widget key is new
+    st.session_state.nonce += 1  # ensures fresh widget keys
 
 def set_scenario(name):
     st.session_state.scenario = name
@@ -262,19 +241,16 @@ def all_loops_complete_for_current_scenario() -> bool:
 # Diagram (arrows removed)
 # ==============================
 def diagram_html(sensor_txt, control_txt, effector_txt):
-    # Box geometry
     sx, sy, sw, sh = 110, 260, 240, 56     # Sensor (left)
     cx, cy, cw, ch = 420, 70,  220, 56     # Control (top)
     ex, ey, ew, eh = 740, 320, 260, 56     # Effector (right)
 
-    # Baseline
     base_x, base_y, base_w, base_h = 330, 520, 360, 26
     base_mid_x = base_x + base_w/2
 
     html = f"""
     <div style="position:relative;width:100%;background:#f6f5ff;border:1px solid #e3e3f8;border-radius:16px;overflow:hidden;">
       <svg viewBox="0 0 1000 620" style="width:100%;height:auto;display:block" preserveAspectRatio="xMidYMid meet">
-        <!-- Baseline -->
         <rect x="{base_x}" y="{base_y}" width="{base_w}" height="{base_h}" rx="8"
               fill="none" stroke="#4b2bb3" stroke-width="5"/>
         <polygon points="{base_mid_x - 20},{base_y + base_h + 2}
@@ -286,7 +262,6 @@ def diagram_html(sensor_txt, control_txt, effector_txt):
         <text x="{base_x + base_w - 70}" y="{base_y + base_h - 8}" font-size="16"
               text-anchor="middle" fill="#4b2bb3">Balance</text>
 
-        <!-- Boxes (no arrows) -->
         <rect x="{sx}" y="{sy}" width="{sw}" height="{sh}" rx="12" fill="#e9ecff" stroke="#6b57e5"/>
         <text x="{sx+sw/2}" y="{sy+sh/2+6}" font-size="16" text-anchor="middle" fill="#1f1d2e">{sensor_txt}</text>
 
@@ -343,7 +318,6 @@ def init_and_render():
 
     st.title("🧩 Interdependence Concept Game")
 
-    # Scenario select (reruns on change)
     scenario_list = list(SCENARIOS.keys())
     idx = scenario_list.index(st.session_state.scenario)
 
@@ -359,25 +333,24 @@ def init_and_render():
             reset_loop()
             safe_rerun()
 
-    # Current loop & gating tokens/keys
     cloop = current_loop()
     token = loop_token()
     ksfx = key_suffix()
 
     st.subheader(f"Stage 1 · Build the negative feedback loop — **{cloop['loop_name']}**")
 
-    # Diagram render (no arrows) — give it a unique key so it refreshes cleanly
+    # Diagram render (no arrows)
     labels = st.session_state.assign
     html = diagram_html(
         labels['sensor'] or "Sensor",
         labels['control'] or "Control Center",
         labels['effector'] or "Effector(s)"
     )
-    components.html(html, height=660, scrolling=False, key=f"diagram_{ksfx}")
+    components.html(html, height=660, scrolling=False)
 
     st.markdown("---")
 
-    # ------- Stage 1 (only this stage shows until passed) -------
+    # ------- Stage 1 -------
     st.write("**Sensor options**")
     sel_s = st.radio("Sensor", cloop["sensor_options"], index=None, horizontal=True, key=f"s1_sensor_{ksfx}")
     if sel_s is not None: st.session_state.assign["sensor"] = sel_s
@@ -392,24 +365,24 @@ def init_and_render():
 
     if st.button("Check Stage 1", key=f"chk1_{ksfx}"):
         a = st.session_state.assign
-        missing = [k for k,v in a.items() if not v]
-        if missing:
-            st.session_state.msgs["s1"] = "Place all three answers."
+        if not all([a["sensor"], a["control"], a["effector"]]):
+            st.session_state.msgs["s1"] = "Please complete all answers."
         else:
-            wrong = []
-            if a["sensor"]  != cloop["sensor_correct"]:  wrong.append("Sensor")
-            if a["control"] != cloop["control_correct"]: wrong.append("Control Center")
-            if a["effector"]!= cloop["effector_correct"]:wrong.append("Effector(s)")
-            if wrong:
-                st.session_state.msgs["s1"] = "Not quite. Check: " + ", ".join(wrong)
-            else:
+            ok = (
+                a["sensor"]  == cloop["sensor_correct"] and
+                a["control"] == cloop["control_correct"] and
+                a["effector"]== cloop["effector_correct"]
+            )
+            if ok:
                 st.session_state.msgs["s1"] = "Great! Proceed to Stage 2."
                 st.session_state.stage = 2
-                st.session_state.stage_token = token   # unlock later stages for THIS loop only
+                st.session_state.stage_token = token
                 safe_rerun()
+            else:
+                st.session_state.msgs["s1"] = "Please recheck your answers."
     st.info(st.session_state.msgs["s1"])
 
-    # Helper: show only one stage at a time and only for this loop
+    # Only show one stage at a time, and only for this loop
     def show_stage(n: int) -> bool:
         return (st.session_state.stage == n) and (st.session_state.stage_token == token if n >= 2 else True)
 
@@ -422,16 +395,12 @@ def init_and_render():
         grd = st.radio("Gradient (primary transport)", ["Concentration","Electrochemical","Pressure"], index=None, horizontal=True, key=f"st2_grad_{ksfx}")
 
         if st.button("Check Stage 2", key=f"chk2_{ksfx}"):
-            ok = (sig==cloop["stage2_sig"] and rec==cloop["stage2_rec"] and grd==cloop["stage2_grad"])
-            if ok:
+            if sig==cloop["stage2_sig"] and rec==cloop["stage2_rec"] and grd==cloop["stage2_grad"]:
                 st.session_state.msgs["s2"] = "Correct! Continue to Stage 3."
                 st.session_state.stage = 3
                 safe_rerun()
             else:
-                msg = "Check again."
-                if "stage2_feedback_wrong" in cloop:
-                    msg += " " + cloop["stage2_feedback_wrong"]
-                st.session_state.msgs["s2"] = msg
+                st.session_state.msgs["s2"] = "Please recheck your answers."
         st.info(st.session_state.msgs["s2"])
 
     # ------- Stage 3 -------
@@ -443,16 +412,12 @@ def init_and_render():
         grd3 = st.radio("Gradient (primary transport)", ["Concentration","Electrochemical","Pressure"], index=None, horizontal=True, key=f"st3_grad_{ksfx}")
 
         if st.button("Check Stage 3", key=f"chk3_{ksfx}"):
-            ok = (sig3==cloop["stage3_sig"] and rec3==cloop["stage3_rec"] and grd3==cloop["stage3_grad"])
-            if ok:
+            if sig3==cloop["stage3_sig"] and rec3==cloop["stage3_rec"] and grd3==cloop["stage3_grad"]:
                 st.session_state.msgs["s3"] = "Nice! Final question…"
                 st.session_state.stage = 4
                 safe_rerun()
             else:
-                msg = "Try again."
-                if "stage3_feedback_wrong" in cloop:
-                    msg += " " + cloop["stage3_feedback_wrong"]
-                st.session_state.msgs["s3"] = msg
+                st.session_state.msgs["s3"] = "Please recheck your answers."
         st.info(st.session_state.msgs["s3"])
 
     # ------- Stage 4 -------
@@ -465,12 +430,12 @@ def init_and_render():
                 st.session_state.msgs["s4"] = "✅ Correct."
                 next_loop_or_finish()  # resets & advances; only Stage 1 will show next
             else:
-                st.session_state.msgs["s4"] = "❌ Not quite—revisit the loop."
+                st.session_state.msgs["s4"] = "Please recheck your answers."
         st.info(st.session_state.msgs["s4"])
 
     st.markdown("---")
 
-    # Certificate (per-scenario after both loops complete)
+    # Certificate
     if all_loops_complete_for_current_scenario() and REPORTLAB_AVAILABLE:
         st.success("Scenario complete. Generate your PDF certificate below.")
         student_name = st.text_input("Student name for certificate", "")