Update explainers/explainer_percent.py

explainers/explainer_percent.py

@@ -1,5 +1,3 @@
-# explainer_percent.py
-
 import re
 from models import ExplainerResult
 
@@ -22,7 +20,7 @@ def explain_percent_question(text: str) -> ExplainerResult | None:
     if percent_matches:
         givens.append(f"A percentage is given: {', '.join(percent_matches[:3])}")
     else:
-        givens.append("A percent relationship is given")
+        givens.append("A percent relationship is described")
 
     if "of" in t:
         givens.append("The wording uses an 'of' relationship")
@@ -32,38 +30,23 @@ def explain_percent_question(text: str) -> ExplainerResult | None:
 
     relationships.append("This is a part-percent-whole relationship")
 
-    if "of" in t:
-        relationships.append("One quantity is being described as a percent of another quantity")
-
     if "increase" in t:
-        relationships.append("The question may involve a percent increase from an original value")
+        relationships.append("This may involve a percent increase from an original value")
 
     if "decrease" in t:
-        relationships.append("The question may involve a percent decrease from an original value")
+        relationships.append("This may involve a percent decrease from an original value")
 
-    if "original" in t or "whole" in t or "base" in t:
-        constraints.append("Pay attention to which value is the original whole or base")
+    if "original" in t or "base" in t:
+        constraints.append("Identify the original/base value carefully")
 
-    if "more than" in t or "less than" in t:
-        constraints.append("Watch the comparison wording carefully")
-
-    asks_for = None
-    if "original" in t or "whole" in t:
-        asks_for = "the original whole value"
-    elif "what percent" in t or "what percentage" in t:
-        asks_for = "the missing percent"
-    elif "what is" in t or "find" in t or "how much" in t:
-        asks_for = "the missing value in the percent relationship"
+    asks_for = "the missing value in the percent relationship"
 
     trap_notes.extend([
         "Do not confuse the part with the whole",
-        "Check whether you are solving for the percent, the part, or the whole",
-        "Make sure the percent is attached to the correct base value",
+        "Check whether you are solving for percent, part, or whole",
+        "Make sure the percent is applied to the correct base value",
     ])
 
-    if "increase" in t or "decrease" in t:
-        trap_notes.append("Do not mix up the original amount and the changed amount")
-
     return ExplainerResult(
         understood=True,
         topic="percent",
@@ -73,10 +56,10 @@ def explain_percent_question(text: str) -> ExplainerResult | None:
         relationships=relationships,
         needed_concepts=[
             "percent equation",
-            "part-whole thinking",
-            "identifying the base value",
+            "part = percent × whole",
+            "base value identification",
         ],
         trap_notes=trap_notes,
-        strategy_hint="First decide which number is the whole, which is the part, and whether the question is asking you to work forward or backward.",
-        plain_english="The question is describing a percent relationship and asking you to identify the missing value in that setup.",
+        strategy_hint="Translate the sentence into a percent equation before calculating anything.",
+        plain_english="The question is describing a percent relationship and asking you to identify a missing value within that structure.",
     )