README describes LLM summarization that isn't wired in #4
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Category: Docs · Priority: MED
Summary
README says
summarize.py"uses Ollama llama3.2:1b when present." Actual v1 is deterministic regex extraction (no LLM); the LLM path is a future--llmflag, not part of the daily cron. Ollama is installed but not running and no model is pulled.Why it matters
Misleads operators about what's actually running; complicates debugging and erodes trust in the pipeline description.
What happens if not done
Future maintainers waste time expecting model output that never appears; misconfigured expectations around cost/latency.
Suggested approach
Update README to state v1 = deterministic extraction, LLM mode = future opt-in
--llmflag, currently disabled. Reflect actualENABLED_SOURCES.Resolved. README updated to reflect actual state: v1 summarize.py = deterministic regex extraction (no LLM). Sub-agent summarization cron (
athena-summarize-batch, every 4h) now provides LLM-quality summaries for high-signal entries, writing JSON back toentries.summary. Generator (generate_from_athena.py) wired to displayone_linerunder each headline on the live CFP feed.Resolved. README updated to reflect actual state: v1 summarize.py = deterministic regex extraction (no LLM). Sub-agent summarization cron (athena-summarize-batch, every 4h) now provides LLM-quality summaries for high-signal entries. Generator wired to display one_liner under each headline.