Skip to content

chore: update sglang to chtruong814 fork#2246

Draft
kajalj22 wants to merge 3 commits intomainfrom
update-sglang-chtruong-fork
Draft

chore: update sglang to chtruong814 fork#2246
kajalj22 wants to merge 3 commits intomainfrom
update-sglang-chtruong-fork

Conversation

@kajalj22
Copy link
Copy Markdown
Contributor

@kajalj22 kajalj22 commented Apr 10, 2026

Summary

  • Switch sglang dependency from JustinTong0323/sglang to chtruong814/sglang (update-transformers-v5 branch)
  • Update commit hash to latest: 78135940075cd87307f6a764d4402188d00ef7c3
  • Update dependency metadata to reflect transformers version change (5.3.0 → 5.2.0 in metadata, though global override still enforces 5.3.0)

🤖 Generated with Claude Code

@kajalj22 kajalj22 requested a review from a team as a code owner April 10, 2026 04:31
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Apr 10, 2026

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.

Switch sglang dependency from JustinTong0323/sglang to chtruong814/sglang
(update-transformers-v5 branch) to use the latest upstream changes.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Kajal Jain <kajalj@nvidia.com>
@kajalj22 kajalj22 force-pushed the update-sglang-chtruong-fork branch from 8f203da to efe4b3f Compare April 10, 2026 04:32
This PR upgrades Python from 3.13.11 to 3.13.13 and uv from 0.11.3 to 0.11.6 across the NeMo-RL codebase.

Changes Made:
- Updated `.python-version` to 3.13.13
- Updated `pyproject.toml` `requires-python` to `>=3.13.13`
- Updated `Dockerfile` to use Python 3.13.13 and uv 0.11.6
- Updated `Dockerfile.ngc_pytorch` to use uv 0.11.6
- Updated `pyrefly.toml` python-version to 3.13.13
- Updated `.claude/skills/code-style/SKILL.md` to Python 3.13.13+
- Updated `research/template_project/` files to Python 3.13.13
- Regenerated `uv.lock` with Python 3.13.13

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Kajal Jain <kajalj@nvidia.com>
@kajalj22 kajalj22 requested review from a team and terrykong as code owners April 10, 2026 04:34
Add override dependency to upgrade pytest to 9.0.3.

Changes:
- Added pytest>=9.0.3 to override-dependencies
- Updated in uv.lock: v9.0.2 → v9.0.3

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Kajal Jain <kajalj@nvidia.com>
@kajalj22
Copy link
Copy Markdown
Contributor Author

/ok to test b7e0638

@kajalj22 kajalj22 added the CI:L1 Run doctests, unit tests, and functional tests label Apr 10, 2026
@kajalj22 kajalj22 marked this pull request as draft April 10, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:L1 Run doctests, unit tests, and functional tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant