Skip to content

Run The Sunken Crypt on three decision models, and say what the laten… - #5

Merged
samkeen merged 1 commit into
mainfrom
godot-jev-three-models
Oct 8, 2026
Merged

samkeen merged 1 commit into
mainfrom
godot-jev-three-models

Conversation

@samkeen

@samkeen samkeen commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

…cies measure

godot-jev: compare_backends.gd takes extra Open Jev servers with --openjev name=url, records how many options each question offered and the thresholds used, and counts agreement for every pair of backends. The 16 session lines were run for 3 rounds on TypeSafe Jev (jev-1.13.0), Open Jev 27B and Open Jev Flash 9B: all three landed 48 of 48 with no wrong move, and Flash 9B, being less sure, made the game ask on two lines. The run is written up in eval/results-2026-10-08.md and recorded under check 2 in the README. Both result files it rests on are committed, including the 2026-09-30 run it is compared with.

docs: a godot-jev results page in the style of the model-router one. Its data comes from docs/tools/build-godot-jev-data.gd, which replays every recorded answer through the game's own parser, at the run's thresholds and at each setting the page's sliders reach, and refuses to write if an outcome or a summary differs from the run. The styles, tooltips, scoreboard and latency chart the two results pages share move to docs/assets/results.css and results.js.

Latency: the local models' timings are from 4-bit MLX builds on one laptop running macOS with other applications open, and the hosted ones include the network. The landing page, both results pages and the READMEs now say that these are readings of a setup, not of how fast the models are.

…cies measure

godot-jev: compare_backends.gd takes extra Open Jev servers with
`--openjev name=url`, records how many options each question offered and
the thresholds used, and counts agreement for every pair of backends. The
16 session lines were run for 3 rounds on TypeSafe Jev (jev-1.13.0), Open
Jev 27B and Open Jev Flash 9B: all three landed 48 of 48 with no wrong
move, and Flash 9B, being less sure, made the game ask on two lines. The
run is written up in eval/results-2026-10-08.md and recorded under check 2
in the README. Both result files it rests on are committed, including the
2026-09-30 run it is compared with.

docs: a godot-jev results page in the style of the model-router one. Its
data comes from docs/tools/build-godot-jev-data.gd, which replays every
recorded answer through the game's own parser, at the run's thresholds and
at each setting the page's sliders reach, and refuses to write if an
outcome or a summary differs from the run. The styles, tooltips,
scoreboard and latency chart the two results pages share move to
docs/assets/results.css and results.js.

Latency: the local models' timings are from 4-bit MLX builds on one
laptop running macOS with other applications open, and the hosted ones
include the network. The landing page, both results pages and the READMEs
now say that these are readings of a setup, not of how fast the models
are.

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

samkeen commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

LGTM

@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: fd23e8e2-61a9-4d8c-ba28-0a7cfa52b35d
📥 Commits

Reviewing files that changed from the base of the PR and between fd324d2 and 4483622.

⛔ Files ignored due to path filters (1)
  • docs/godot-jev/screenshot.png is excluded by !**/*.png
📒 Files selected for processing (19)
  • README.md
  • docs/README.md
  • docs/assets/results.css
  • docs/assets/results.js
  • docs/godot-jev/app.js
  • docs/godot-jev/data.js
  • docs/godot-jev/index.html
  • docs/godot-jev/page.css
  • docs/index.html
  • docs/model-router/app.js
  • docs/model-router/index.html
  • docs/tools/build-godot-jev-data.gd
  • godot-jev/README.md
  • godot-jev/eval/compare.gd
  • godot-jev/eval/compare_backends.gd
  • godot-jev/eval/results-2026-09-30T101313.json
  • godot-jev/eval/results-2026-10-08.md
  • godot-jev/eval/results-2026-10-08T134205.json
  • godot-jev/tests/test_compare.gd
 __________________________________________________________
< Keep your friends close, but your code reviewers closer. >
 ----------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@samkeen
samkeen merged commit aebf045 into main Oct 8, 2026
5 of 6 checks passed
@samkeen
samkeen deleted the godot-jev-three-models branch October 8, 2026 21:41
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