Skip to content

Update GLM 5 B200 FP8 serve command - add SGLANG_ENABLE_JIT_DEEPGEMM - #27598

Closed
faradawn wants to merge 4 commits into
sgl-project:mainfrom
faradawn:update-glm-5
Closed

Update GLM 5 B200 FP8 serve command - add SGLANG_ENABLE_JIT_DEEPGEMM#27598
faradawn wants to merge 4 commits into
sgl-project:mainfrom
faradawn:update-glm-5

Conversation

@faradawn

@faradawn faradawn commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Motivation

Add SGLANG_ENABLE_JIT_DEEPGEMM flag.

Reference: SemiAnalysisAI/InferenceX#1567

Modifications

Accuracy Tests

Speed Tests and Profiling

Checklist

Review and Merge Process

  1. Ping Merge Oncalls to start the process. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments or contact authorized users to do so.
    • Common commands include /tag-and-rerun-ci, /tag-run-ci-label, /rerun-failed-ci
  4. After green CI and required approvals, ask Merge Oncalls or people with Write permission to merge the PR.

CI States

Latest PR Test (Base): ✅ Run #27167779141
Latest PR Test (Extra): ❌ Run #27167779032

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Jun 8, 2026

@b8zhong b8zhong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think DeepGEMM is enabled by default, so this flag doesn't have an effect. Is this just for consistency in cookbook InfX submissions?

@faradawn

faradawn commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

If it is enabled by default, then we are good!

@b8zhong

b8zhong commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

@b8zhong

b8zhong commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

@faradawn Let's close this for now. Since DeepGEMM is already enabled by default (it might be better to just remove it in the InfX recipe)

@b8zhong b8zhong closed this Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants