Skip to content

fix(rollout): make dynamic refills granular#2217

Closed
EazyReal wants to merge 1 commit into
THUDM:mainfrom
EazyReal:maxwill/upstream-granular-rollout-refill
Closed

fix(rollout): make dynamic refills granular#2217
EazyReal wants to merge 1 commit into
THUDM:mainfrom
EazyReal:maxwill/upstream-granular-rollout-refill

Conversation

@EazyReal

@EazyReal EazyReal commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Problem

Dynamic sampling currently requires over_sampling_batch_size >= rollout_batch_size, even though refill generation can safely request only the number of groups still missing. This forces oversized refill batches.

Stopping a non-partial rollout also waits for pending agent tasks until after SGLang is drained. Those tasks can keep issuing requests while the drain is in progress, delaying shutdown.

Change

  • accept any positive over_sampling_batch_size
  • cancel and await pending agent tasks before draining SGLang for non-partial rollouts
  • preserve partial-rollout collection behavior
  • register focused CPU coverage for refill validation and both abort modes

Allow positive refill sizes smaller than the target rollout batch, and cancel unfinished non-partial generation tasks before waiting for SGLang to drain. This prevents one rejected prompt group from launching a full replacement wave or keeping the drain loop alive through new agent turns.
@EazyReal
EazyReal force-pushed the maxwill/upstream-granular-rollout-refill branch from f6f503e to e3eca96 Compare July 18, 2026 02:14
@EazyReal EazyReal closed this Jul 18, 2026
@EazyReal
EazyReal deleted the maxwill/upstream-granular-rollout-refill branch July 18, 2026 02:30
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