helmo Codex commited on
Commit
407af0d
·
1 Parent(s): 644e47e

[docs] Neutral comments: drop self-promotion framing; describe behaviour, not the model

Browse files

Comment-only cleanup. The intake attribution comments no longer read like the
app's purpose is marketing the author (removed 'carries the other follow links…'
etc.) — they now just say what the element is. The _is_weak_reasoning docstring
describes the unusable-response condition rather than labelling 'the model'.
No behaviour change.

Co-authored-by: Codex <noreply@openai.com>

Files changed (2) hide show
  1. app/phases/assessment.py +4 -6
  2. app/phases/intake.py +3 -5
app/phases/assessment.py CHANGED
@@ -162,14 +162,12 @@ _GROUND_MAP = {
162
 
163
 
164
  def _is_weak_reasoning(text: str, result) -> bool:
165
- """Deterministic test for a non-answer — no brittle prose pattern-matching.
166
 
167
  The assessment is required to end with a structured block (case_type / grounds
168
- / countries). The model is weak/off-task when it produced **none** of that
169
- structure, or barely any narration. When that happens (e.g. it drifted into a
170
- chatbot greeting because the context window truncated its instructions), the
171
- interview-grounded analysis takes over. With an adequate context window this
172
- rarely fires."""
173
  if len((text or "").strip()) < 120:
174
  return True
175
  return not (result.case_type or result.grounds or result.countries)
 
162
 
163
 
164
  def _is_weak_reasoning(text: str, result) -> bool:
165
+ """Deterministic test for a non-answer — no prose pattern-matching.
166
 
167
  The assessment is required to end with a structured block (case_type / grounds
168
+ / countries). The response is treated as unusable when it contains **none** of
169
+ that structure, or barely any narration and the interview-grounded analysis
170
+ takes over instead. With an adequate context window this rarely fires."""
 
 
171
  if len((text or "").strip()) < 120:
172
  return True
173
  return not (result.case_type or result.grounds or result.countries)
app/phases/intake.py CHANGED
@@ -75,8 +75,7 @@ INTAKE_CSS = """
75
  padding:9px 16px; border-radius:var(--r-full); }
76
  .intake-trust svg { flex:0 0 auto; color:var(--primary); }
77
 
78
- /* Hackathon credit first screen only. @helmo links to the HF profile, which
79
- carries the other follow links (LinkedIn, etc.). */
80
  #intake-credit { margin-top:22px; text-align:center; font-size:12px; color:var(--text-muted); }
81
  #intake-credit a { color:var(--primary-deep); text-decoration:none; font-weight:600; }
82
  #intake-credit a:hover { color:var(--primary); text-decoration:underline; }
@@ -119,8 +118,7 @@ _TRUST_HTML = """
119
  </span></div>
120
  """
121
 
122
- # Hackathon credit first screen only. @helmo points to the HF profile, which
123
- # carries the rest of the follow links.
124
  _CREDIT_HTML = (
125
  '<div id="intake-credit">Made by '
126
  '<a href="https://huggingface.co/helmo" target="_blank" rel="noopener noreferrer">@helmo</a> '
@@ -189,7 +187,7 @@ def build(visible: bool = True) -> IntakeUI:
189
  with gr.Row(elem_classes=["intake-cta"]):
190
  begin = gr.Button("Begin", elem_id="intake-begin", interactive=False)
191
  gr.HTML(_TRUST_HTML)
192
- gr.HTML(_CREDIT_HTML) # author credit below the card, intake screen only
193
 
194
  def _on_pill(clicked: str, current: str | None):
195
  new_selected = toggle_selection(clicked, current)
 
75
  padding:9px 16px; border-radius:var(--r-full); }
76
  .intake-trust svg { flex:0 0 auto; color:var(--primary); }
77
 
78
+ /* Small hackathon attribution line, shown on the first screen only. */
 
79
  #intake-credit { margin-top:22px; text-align:center; font-size:12px; color:var(--text-muted); }
80
  #intake-credit a { color:var(--primary-deep); text-decoration:none; font-weight:600; }
81
  #intake-credit a:hover { color:var(--primary); text-decoration:underline; }
 
118
  </span></div>
119
  """
120
 
121
+ # Hackathon attribution line, shown on the first screen only.
 
122
  _CREDIT_HTML = (
123
  '<div id="intake-credit">Made by '
124
  '<a href="https://huggingface.co/helmo" target="_blank" rel="noopener noreferrer">@helmo</a> '
 
187
  with gr.Row(elem_classes=["intake-cta"]):
188
  begin = gr.Button("Begin", elem_id="intake-begin", interactive=False)
189
  gr.HTML(_TRUST_HTML)
190
+ gr.HTML(_CREDIT_HTML) # attribution line, below the card (intake only)
191
 
192
  def _on_pill(clicked: str, current: str | None):
193
  new_selected = toggle_selection(clicked, current)