Skip to content

docs: architecture + comparison — first-round fact pass - #120

Merged
cherusk merged 1 commit into
mainfrom
docs/architecture-facts
Sep 6, 2026
Merged

docs: architecture + comparison — first-round fact pass#120
cherusk merged 1 commit into
mainfrom
docs/architecture-facts

Conversation

@godon-robot

@godon-robot godon-robot Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

FIRST ROUND — scoped to the audited pages; api.md not yet verified. Held open for review.

  • Worker groups: replica counts column removed entirely rather than refreshed (they had already gone stale once: 5/2 vs actual 10/3). Replica counts are deployment values living in the chart's values.yaml, not architecture. The timeout column stays qualitative — "none by design" on breeder workers (crash recovery via the Optuna DB) is the architectural fact; "2 minutes" was also a chart value.
  • Cooperation scaling: the unsourced "diminishing returns after ~10 cooperating breeders" claim is dropped — nothing in engine or notes backs it.
  • Causal endpoints: completed to the verified route list from godon-causal main.rs — adds /predict/multihop, /graph, /artifact (the exportable measured map), /walk-view/{sender}, /impact/{breeder_id}, /causes/{breeder_id}.
  • Comparison: effectuation claims trimmed to shipped reality (SSH, HTTP — the "APIs" rode on the unshipped Kubernetes effectuator); the Optuna reconnaissance row gains HTTP (Prometheus, HTTP — both shipped).

Worker groups: replica counts removed — deployment values belong in
values.yaml, not architecture (they had already gone stale once); the
timeout column stays qualitative (breeder no-timeout-by-design is the
architectural fact). Unsourced 'diminishing returns after ~10
cooperating breeders' claim dropped. Causal endpoints completed to the
verified route list (/predict/multihop, /graph, /artifact, /walk-view,
/impact, /causes). Comparison: effectuation claims trimmed to shipped
SSH+HTTP; Optuna recon row gains HTTP.

First round — scoped to the audited pages; api.md not yet verified.

Co-authored-by: cherusk <10729954+cherusk@users.noreply.github.com>
@cherusk
cherusk merged commit dc7cc7b into main Sep 6, 2026
2 checks passed
@cherusk
cherusk deleted the docs/architecture-facts branch September 6, 2026 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant