Skip to content

tmp: verify #229 with Tests workflow from #228 (do not merge) - #230

Closed
jdueadev wants to merge 3 commits into
mainfrom
tmp/verify-test-fixes
Closed

jdueadev wants to merge 3 commits into
mainfrom
tmp/verify-test-fixes

Conversation

@jdueadev

Copy link
Copy Markdown
Owner

Temporary draft that combines #228 and #229 so the Linux make test-fast job runs against the fixes. I'll close it once CI reports.

🤖 Generated with Claude Code

jdueadev and others added 3 commits November 12, 2026 17:09
Claude review runs on the owner's Claude subscription via
CLAUDE_CODE_OAUTH_TOKEN. CodeRabbit and Gemini Code Assist configs take
effect once their GitHub Apps are installed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Restore scripts/kaggle_worker_entry.py and scripts/kaggle_runtime_env.py.
  6d9fe7f removed them as obsolete, but the Kaggle/Colab launchers
  (kaggle_runner, kernel_package, remote_package, colab_runner) still
  package and run them, and benchmark_jax_rl imports kaggle_runtime_env.
- rollout-phase-profile: SystemExit() takes no `file=` kwarg, so a missing
  integration worktree crashed with TypeError instead of a clean message.
- Kaggle submission packager: add factored_decode_scan, map_pool/load and
  map_pool/bake to RUNTIME_FILES; packaged modules import them, so the
  submission would fail with ModuleNotFoundError.
- Tests: src.training.curriculum moved to src.opponents.curriculum;
  bounds_exit_time test used a 1-unit board (BOARD_SIZE is 100);
  preflight sweep reseed_every_updates was deliberately raised to 100.
- Docs: drop links to files removed at retirement and to uncommitted
  outputs/, and reword AGENTS.md so it no longer names removed doc dirs.

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

Copy link
Copy Markdown
Owner Author

Verification done: make test-fast on Linux reports 1344 passed and 0 failed with #229's fixes, using the Tests workflow from #228. Closing this temporary PR.

@jdueadev jdueadev closed this Nov 13, 2026
@jdueadev
jdueadev deleted the tmp/verify-test-fixes branch November 13, 2026 00:17
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