Skip to content

openspec: S24 sampler island, trainer start/stop, decoupled PPO, identity tiering - #258

Draft
michaellchung wants to merge 2 commits into
mainfrom
s24-sampler-spec
Draft

michaellchung wants to merge 2 commits into
mainfrom
s24-sampler-spec

Conversation

@michaellchung

Copy link
Copy Markdown
Contributor

Summary

Adds the OpenSpec change rl-trainer-cold-start-cost (proposal, design, tasks, spec delta). Spec only; no code changes.

Goal: lower the start cost of a trainer island, so that a trainer can start and stop often (S24 sampler island work).

  • L0: seven start phase events; the scheduler reads the start time T_s from them.
  • L2: Miles import chain fix (light actors stop importing miles.backends.megatron_utils; lazy modelopt import), setup reads the image manifest instead of importing peft/sglang, peft 0.20.0 in the ports image, replace the broken huggingface-cli download prefetch and record failures.
  • L1.1: measure a separate trainer island in two steps: Qwen3.5-4B on 2 cards (TP2) first, then decide on Qwen3.8-27B on 8 cards.
  • L8: SGLang memory snapshot for samplers only, gated on a GPU check that it works with RL weight updates.

All effort and GPU cost numbers in tasks.md are estimates. Nothing has run on GPU for this change yet.

Validation

  • openspec validate rl-trainer-cold-start-cost: valid.

🤖 Generated with Claude Code

chowhsu and others added 2 commits October 14, 2026 10:59
Proposal, design, tasks and spec delta to cut trainer start cost:
fix the Miles import chain, read the image manifest in setup, replace
the broken prefetch, add start phase events (T_s), and gate the
SGLang snapshot on a weight-update compatibility check. L1.1 measures
a separate trainer island in two steps (Qwen3.5-4B 2 cards first).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…identity tiering

Record decisions D1-D4 and update task status with GPU evidence.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@michaellchung michaellchung changed the title openspec: rl-trainer-cold-start-cost openspec: S24 sampler island, trainer start/stop, decoupled PPO, identity tiering Oct 15, 2026
@michaellchung

Copy link
Copy Markdown
Contributor Author

Superseded by the stacked draft PRs #262 (cold start) → #263 (off-policy correction) → #264 (identity tiering) → #265 (start/stop) → #266 (TB2/27B run support). Each implementation PR carries its own openspec change. Keeping this one open for reference until the stack is reviewed.

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.

2 participants