Skip to content

Fix ruff-format failure in sandbox rollout observability changes#1928

Draft
Copilot wants to merge 4 commits into
mainfrom
copilot/fix-lint-job-failure
Draft

Fix ruff-format failure in sandbox rollout observability changes#1928
Copilot wants to merge 4 commits into
mainfrom
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Jun 22, 2026

Copy link
Copy Markdown

The lint GitHub Actions job was failing on ruff-format in the sandbox rollout observability changes. This PR aligns the affected expression in sandbox.py with the repository’s enforced formatter output.

  • Scope

    • Update the formatting of the sandbox rate-limit wait-time metadata assignment in xtuner/v1/rl/agent_loop/sandbox_agent_loop/sandbox.py
    • No behavioral or control-flow changes
  • Change

    • Reflow the sandbox_acquire_rate_limit_wait_s accumulation expression to the single-line form required by Ruff
  • Resulting code

record.metadata["sandbox_acquire_rate_limit_wait_s"] = (
    record.metadata.get("sandbox_acquire_rate_limit_wait_s", 0.0) + time.monotonic() - t_limit
)

Copilot AI changed the title [WIP] Fix failing GitHub Actions job lint Fix ruff-format failure in sandbox rollout observability changes Jun 22, 2026
Copilot AI requested a review from Harold-lkk June 22, 2026 16:42
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