Skip to content

perf(checkpoint): reduce Gym control-plane overhead - #4321

Open
macandro96 wants to merge 1 commit into
amahishi/prefix-recovery-hardeningfrom
amahishi/rl-checkpoint-scale-optimizations
Open

macandro96 wants to merge 1 commit into
amahishi/prefix-recovery-hardeningfrom
amahishi/rl-checkpoint-scale-optimizations

Conversation

@macandro96

@macandro96 macandro96 commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • return and consume completion receipts inline with /run
  • batch and retry idempotent completion acknowledgements with reserved control-plane HTTP capacity
  • coalesce controller ACK obligations without holding the data-plane mutation barrier during network I/O
  • commit independent Gym participants concurrently within dependency-ordered stages

Stack

Validation

  • ruff check passed on the reconciled optimization files
  • Python compilation passed
  • scale benchmarks and cluster checkpoint suites still need to run

@copy-pr-bot

copy-pr-bot Bot commented Sep 29, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

✅ Submodule Fast-Forward Check Results

Check based on commit: 411df78 (PR #4321 from amahishi/rl-checkpoint-scale-optimizations)

✅ Submodules that are properly updated:

Gym: ✅ PR branch is ahead of amahishi/prefix-recovery-hardening branch (fast-forward)

All submodule changes look good! ✨

@macandro96
macandro96 added this pull request to stack #4267 September 29, 2026 04:25
@macandro96
macandro96 marked this pull request as ready for review September 29, 2026 04:28
Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
@macandro96
macandro96 force-pushed the amahishi/rl-checkpoint-scale-optimizations branch from 411df78 to 3dd6077 Compare September 29, 2026 04:45

This branch has not been deployed

No deployments
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