Repository navigation
Docs: point to the quickstart repository (#563) - #568
Conversation
There was a problem hiding this comment.
Round 1 — reviewed head 7ee434ac — reviewer hermes/gpt-5.6-terra.
terra
Advisory findings from outerloop — the code owner decides. Reply to disagree; the outerloop:no-review label opts this PR out.
Verdict: nothing blocking — 2 advisory notes.
2 findings attached to the lines below.
The linked quickstart repository currently requires GitHub, contrary to both new documentation claims.
There was a problem hiding this comment.
Round 2 (re-run on the same head) — reviewed head 7ee434ac — reviewer summarizer:hermes/gpt-5.6-terra over coverage+credentials+deployment+general+lifecycle+prose.
terra
Advisory findings from outerloop — the code owner decides. Reply to disagree; the outerloop:no-review label opts this PR out.
Verdict: 1 blocking, 4 advisory.
5 findings attached to the lines below.
Merged verdict: one blocking containment/credential exposure, two functional quickstart issues, and two prose suggestions. The GitHub-account contradiction is a single merged finding supported by the general and lifecycle lenses. Rejected: no findings were rejected; coverage reported none.
…ed session and a spend-capped key; plainer wording
There was a problem hiding this comment.
Round 4 (re-run on the same head) — reviewed head a07074e8 — reviewer summarizer:hermes/gpt-5.6-terra over coverage+credentials+deployment+general+lifecycle+prose.
terra
Advisory findings from outerloop — the code owner decides. Reply to disagree; the outerloop:no-review label opts this PR out.
Verdict: nothing blocking — 3 advisory notes.
3 findings attached to the lines below.
Merged non-blocking findings: the quickstart omits Python 3.12 (coverage and general agree), omits a required Claude model ID, and overstates isolation for --no-image. Rejected: none; no opinion was judged mistaken or duplicative beyond the merged Python prerequisite reports.
# Conflicts: # CHANGELOG.md
There was a problem hiding this comment.
Round 5 — reviewed head 925e3348 — reviewer hermes/gpt-5.6-terra.
terra
Advisory findings from outerloop — the code owner decides. Reply to disagree; the outerloop:no-review label opts this PR out.
Verdict: nothing blocking — 2 advisory notes.
2 findings attached to the lines below.
The new quickstart understates its Python requirement and the exposure of uncontained sessions.
…me, account-readable files), not a sandbox; quickstart names Python 3.12
# Conflicts: # CHANGELOG.md
The README's Get started and the getting-started guide now point to outerloop-science/quickstart: two CPU benchmarks run with the local forge and local compute, so there's no GitHub account, no cluster and no container. The README shows the three commands.
The path was tested end to end with
outerloop-science==0.5.0from PyPI on a Linux CPU machine: init, first PR in about 25 minutes,pr show,pr merge, ledger onresearch-log. The quickstart's own README change is outerloop-science/quickstart#1.Written by Claude (Opus 5.5); reviewed by codex, which caught a Codex-author/no-image error and a missing
--no-image, both fixed.