Skip to content

Merge PR#8 moonshot stack into main#9

Merged
kailean merged 1 commit intocopilot/verify-aub-1-0-bpb-code-lengthfrom
copilot/merge-pr-8-into-main
Apr 4, 2026
Merged

Merge PR#8 moonshot stack into main#9
kailean merged 1 commit intocopilot/verify-aub-1-0-bpb-code-lengthfrom
copilot/merge-pr-8-into-main

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 4, 2026

PRs #7 and #8 were never merged to main despite containing the complete moonshot feature stack. This branch brings those changes forward via a clean linear fast-forward.

Changes vs main

  • train_gpt_mlx_kl.py — Full moonshot stack (EngramLite, BackoffNgramMixer, ComplementaryTraining, SkipGramHash, SmearGate, XSA, GPTQ-lite, sliding-window eval, LoRA TTT); trimmed 1847→1493 lines; fixed orphaned clip_grad_tree (body existed, def statement missing — NameError at runtime when grad_clip_norm > 0)
  • pg_novel_ideas.md — Added from brainstorm branch; covers 8 approaches to sub-1.10 BPB with ranked idea list and dead-idea evidence table
  • CLAUDE.md — Fixed broken venv activation path (source ~/pg_env/bin/activate); added moonshot smoke test and full H100 run commands

Merge note

This branch is a clean 5-commit linear chain ahead of main (fast-forward eligible). PR base needs to be retargeted to main before merging, or merge directly:

git checkout main
git merge origin/copilot/merge-pr-8-into-main --no-edit
git push origin main

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh auth status (http block)
  • https://api.github.com/repos/kailean/parameter-golf/pulls/9
    • Triggering command: /usr/bin/curl curl -s -X PATCH -H Accept: application/vnd.github+json -H Authorization: ****** REDACTED -d {"base":"main"} (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Merge PR #8 into main Merge PR#8 moonshot stack into main Apr 4, 2026
Copilot AI requested a review from kailean April 4, 2026 17:46
@kailean kailean marked this pull request as ready for review April 4, 2026 19:02
Copilot AI review requested due to automatic review settings April 4, 2026 19:02
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kailean kailean merged commit be71b98 into copilot/verify-aub-1-0-bpb-code-length Apr 4, 2026
1 check passed
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Apr 6, 2026

🤖 Augment PR Summary

Summary: This PR is intended to fast-forward/merge the complete “moonshot” training stack (plus docs updates) that previously lived in PRs #7/#8.

Review note: GitHub currently shows 0 changed files vs the PR base branch, so there’s no diff to review/annotate until the PR is retargeted (e.g., to main) or recreated with the correct base.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

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.

3 participants