Commit History
-
[NOTICKET] fix: aggregate trend charts + materialize analyze_* tables in the chart chain
A "visualize the trend of X" question produced a broken chart in two ways: render_chart errored ("cannot materialize 'data' of kind 'table'"), and once that was worked around it plotted 9,729 raw rows (a vertical smear) of the wrong column (Plan_PA_Percent aliased "pa_percent") instead of a per-day mean trend. Tools (materialize + render): - _materialize (invoker.py) now accepts the analyze_* table shape: a ToolOutput of kind "table" whose data is `value` = list-of-dicts (columns/rows None), as analyze_aggregate emits. Previously only the retrieve_data columns+rows shape was handled, so ANY analyze_* -> analyze_*/render_chart chain failed. Class fix. - render_chart (visualization.py) resolves a requested x/y column to its aggregated name when the exact name is absent (pa_percent -> pa_percent_mean), so a retrieve -> analyze_aggregate -> render_chart chain doesn't 404 on columns. Planner (planner.md + examples.py): - A line/trend chart over time must produce ONE ROW PER PERI…0f3d8cc
-
fix/ check and new chart tool (#16)
- check: table-level column drill + drop misleading "name a table" hint (ef0c7b1c5aa49a244b02f1b5eea15cc77dcf2bcb) - [KM-715] charts: render_chart tool + message_charts store + GET /api/v1/charts + planner viz slice (87cfcf884d923ee4bcd722e0e4ef4d90d7eaedca) - [KM-715] checkpoint: S1a deterministic quality checkpoint between runner and assembler (f3c8de34dc19a573031b67fd4f48dcfbadcdfed8) - [KM-715] docs: sync REPO_STATUS/DEV_PLAN + land SPINE_V2_PLAN (W1/W2 done) (434eacbf2ac8324188ebe5e32775dbc9d2a3f49a) - planner: value-handoff (${tN.col} → list) for tabular lookup/anti-join (54070ac1407b751000e49dfe7f102a2be5ab1ff8) - [KM-715] charts: GET /charts by message_id only + tri-state status marker (b8d15761addf4d499c9631df64e7693aa1d61fa2) - [KM-715] report: embed charts as plotly EDA fences; render_chart counts toward the floor (89db8be6b6f11e86ad0776d2ee07f2306e3b6079) - [KM-715] docs: contract + status deltas for the charts lead-review round (96347ee7c55a4eb39a906409ec214463e25c5e35) - …5a60e93
