avoigt1121 Claude Opus 4.8 commited on
Commit
f098e67
Β·
1 Parent(s): 5ba3350

docs: gradio 6.18 migration done + DEV RUNNING; SSR fix superseded

Browse files

Records the real fix for the "Starting" hang (gradio 5.49 -> 6.18; SSR was a
red herring), the show_api follow-up, the HF rollout wedge cleared by
pause/unpause, and dev confirmed RUNNING on 5ba3350. Flags prod migration as
the remaining open item (quota-gated). SHOWCASE_STATUS.md updated separately
(it lives outside this repo).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Files changed (2) hide show
  1. TODO.md +21 -4
  2. memory.md +67 -1
TODO.md CHANGED
@@ -12,6 +12,15 @@ pdac-analysis-orchestrator.
12
 
13
  ## Open
14
 
 
 
 
 
 
 
 
 
 
15
  ### Cross-dataset integration (ADR-0001)
16
 
17
  Combine β‰₯2 datasets β€” early (pool + batch covariate) / late (meta-analysis) /
@@ -107,10 +116,18 @@ refuse, driven by registry metadata. ADR + plan + per-task spin-off prompts in
107
 
108
  ## Done (recent)
109
 
110
- - 2026-06-21 β€” Fixed HF Space stuck on "Starting" (`gradio==5.49.0` auto-enables
111
- experimental SSR; HF readiness check hangs). `ssr_mode=False` default in
112
- `GradioAgentUI.launch()` (`d38496a`); pushed to dev (rebuilding) + prod
113
- (staged, still quota-paused). Workaround: Space var `GRADIO_SSR_MODE=false`.
 
 
 
 
 
 
 
 
114
  - 2026-06-17 β€” UI download buttons fixed: exports no longer truncate the
115
  solution (the old path dropped everything after the first blank line); TXT
116
  downloads again; added a **Full log (.pdf)** export of the full generated
 
12
 
13
  ## Open
14
 
15
+ ### Deploy / infra
16
+
17
+ - **High β€” Migrate PROD specialist to gradio 6.18.** Local `main` (`5ba3350`) has
18
+ the verified gradio-6 migration; dev is RUNNING on it. Prod (`origin`) is still
19
+ gradio 5.49 (`d38496a`) and PAUSED on the cpu-basic quota (3/3). To ship: push
20
+ `main` β†’ `origin`, free a slot (e.g. pause `bcc-lit-agent`), then restart/unpause
21
+ the prod Space. Until then prod stays on 5.49 and would hit the same "Starting"
22
+ hang if it ever comes off pause without the migration.
23
+
24
  ### Cross-dataset integration (ADR-0001)
25
 
26
  Combine β‰₯2 datasets β€” early (pool + batch covariate) / late (meta-analysis) /
 
116
 
117
  ## Done (recent)
118
 
119
+ - 2026-06-22 β€” **REALLY fixed HF Space stuck on "Starting": gradio 5.49 β†’ 6.18
120
+ migration** (commits `b4e7ac0` + `5ba3350`, branch `fix/gradio-6-migration` β†’
121
+ merged to `main`). The 2026-06-21 SSR fix was a red herring (the working
122
+ orchestrator runs 6.18 *with SSR on*); real cause is gradio 5.49 no longer
123
+ completing HF's readiness handshake. Cascade: `mcp` 1.10.1β†’1.28.0,
124
+ `langchain-mcp-adapters` 0.2.2β†’0.3.0, `fastmcp` pinned `==3.2.3`; `gradio_ui.py`
125
+ ported to the 6.0 API; `ssr_mode=False` removed. Verified: 851 tests, 49 tools,
126
+ HTTP 200. **DEV RUNNING** on `5ba3350` (a pause→unpause cleared an HF-side
127
+ rollout wedge). Prod migration still pending (see Open β†’ Deploy/infra).
128
+ - 2026-06-21 β€” [SUPERSEDED by the gradio-6 migration above] Attempted SSR fix for
129
+ the "Starting" hang: `ssr_mode=False` default in `GradioAgentUI.launch()`
130
+ (`d38496a`). Did not actually fix it; removed during the migration.
131
  - 2026-06-17 β€” UI download buttons fixed: exports no longer truncate the
132
  solution (the old path dropped everything after the first blank line); TXT
133
  downloads again; added a **Full log (.pdf)** export of the full generated
memory.md CHANGED
@@ -4,7 +4,73 @@ This file tracks current status, recent decisions, and next steps.
4
  Update it whenever meaningful work is completed or the direction changes.
5
  Stable architectural facts belong in `CLAUDE.md`, not here.
6
 
7
- Last updated: 2026-06-21
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
8
 
9
  ---
10
 
 
4
  Update it whenever meaningful work is completed or the direction changes.
5
  Stable architectural facts belong in `CLAUDE.md`, not here.
6
 
7
+ Last updated: 2026-06-22
8
+
9
+ ---
10
+
11
+ ## 2026-06-21/22 session β€” REAL fix for Space stuck on "Starting": gradio 5.49 β†’ 6.18 (SSR was a red herring) β€” DEV RUNNING βœ…
12
+
13
+ The dev specialist Space (`Paper2Agent_decoupleRpy_dev`) was still hanging on
14
+ **APP_STARTING** even after the SSR fix (`d38496a`) was deployed β€” the app bound
15
+ 7860, served, loaded 49 tools, but HF never flipped the badge to Running (0
16
+ ready replicas, public URL 404).
17
+
18
+ **Diagnosis (via HF runtime API, git keychain token):** the Space had been
19
+ wedged building an *old* commit (`c54b550`, pre-SSR-fix); the d38496a/81fb4aa
20
+ pushes updated the repo but never rebuilt the container. Forced a factory
21
+ rebuild β†’ it built `81fb4aa` (SSR fix live) and **still hung on APP_STARTING**.
22
+ Comparing run logs against the known-good `pdac-analysis-orchestrator-dev`
23
+ (RUNNING) showed that Space runs **gradio 6.18 WITH SSR enabled** and reaches
24
+ Running, while this one on **5.49.0** hangs regardless of `ssr_mode`. β†’ Root
25
+ cause: **gradio 5.49.0 no longer completes HF's current Spaces readiness
26
+ handshake.** The SSR commit chased the wrong thing.
27
+
28
+ **Fix (commit `b4e7ac0`, branch `fix/gradio-6-migration`):** migrate to gradio
29
+ 6.18.0.
30
+ - README `sdk_version` 5.49.0 β†’ 6.18.0.
31
+ - requirements: `gradio[oauth,mcp]` 5.49.0 β†’ 6.18.0 lifts the old `mcp==1.10.1`
32
+ hard-pin β†’ `langchain-mcp-adapters` 0.2.2 β†’ 0.3.0, `mcp` β†’ 1.28.0. **`fastmcp`
33
+ pinned ==3.2.3**: 2.11.2 is incompatible with mcpβ‰₯1.28 (`@mcp.tool` returns a
34
+ non-callable FunctionTool β†’ 142 test failures + breaks direct tool calls);
35
+ 3.4.x breaks `from fastmcp import FastMCP`.
36
+ - `gradio_ui.py` ported to the 6.0 API: theme+css moved off `Blocks()` onto
37
+ `launch()`; removed `Chatbot(type=, show_copy_button=)` and `Textbox(
38
+ show_copy_button=)`; dropped the obsolete `ssr_mode=False` override (6.x runs
39
+ with SSR on, like the working Space).
40
+ - **Verified locally** (pinned set): UI builds, 49 MCP tools discover (no
41
+ errors), full suite **851 passed / 55 skipped**.
42
+ - **Second fix needed (`5ba3350`):** the first dev build crash-looped with
43
+ `TypeError: Blocks.launch() got an unexpected keyword argument 'show_api'` β€”
44
+ gradio 6.0 removed `show_api` from `launch()`. Local checks had only built
45
+ `create_app()` + inspected the launch signature, never actually *called*
46
+ `launch()`; fixed and re-verified by really launching (HTTP 200).
47
+ - **HF rollout wedge (infra, not code):** after the `5ba3350` push the dev Space
48
+ got stuck `APP_STARTING`/`rep=0` for hours regardless of config β€” ruled out
49
+ gradio version (6.18 = the RUNNING orchestrator), SSR (tried on AND off via
50
+ `GRADIO_SSR_MODE` var), code (startup log byte-identical to the working
51
+ orchestrator, incl. the benign `__del__` `ValueError`), and quota (had its
52
+ slot). Other cpu-basic Spaces stayed RUNNING β†’ space-specific. A **pause β†’
53
+ unpause** cycle finally cleared it (promoted after the watch window).
54
+ - **DEV CONFIRMED RUNNING** βœ… (`hf-dev` main = `5ba3350`): runtime RUNNING,
55
+ rep=1, authed `https://anne-voigt-paper2agent-decouplerpy-dev.hf.space/` β†’
56
+ HTTP 200 (404 unauthenticated is just the private-Space gate), 49 tools
57
+ loaded, SSR on (Node proxy β†’ Python :7861, same as the orchestrator).
58
+ - **Local `main` fast-forwarded to `5ba3350`.** Branch `fix/gradio-6-migration`
59
+ merged.
60
+ - **Not yet on prod** (`origin`): prod is PAUSED on the cpu-basic quota (3/3:
61
+ dev specialist + dev orchestrator + bcc-lit-agent all RUNNING) and needs a
62
+ freed slot + manual unpause; push `main` β†’ `origin` once a slot is free. The
63
+ two SSR-era prod commits (`d38496a`/`81fb4aa`) are superseded by this
64
+ migration. **`ssr_mode=False` is no longer set in code** β€” 6.x runs with SSR
65
+ on like the working Space.
66
+
67
+ ---
68
+
69
+ ## 2026-06-21 session β€” Fix HF Space stuck on "Starting" (SSR) β€” SUPERSEDED, see entry above
70
+
71
+ NOTE: the SSR theory below was a **red herring** β€” the real cause was the gradio
72
+ 5.49 ↔ HF readiness incompatibility (see the gradio 6.18 migration entry above).
73
+ Kept for history.
74
 
75
  ---
76