fix: update Blackwell log/error messages to include SM12x#18751
Merged
Kangyan-Zhou merged 1 commit intosgl-project:mainfrom Feb 15, 2026
Merged
Conversation
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Three warning/error messages in server_args.py hardcoded "SM100" but the code checks `is_blackwell_supported()` which covers both SM100 and SM12x (consumer Blackwell / DGX Spark). Updated messages to say "Blackwell" or "SM100/SM12x" so users on SM12x devices don't get confused by messages referencing only SM100. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
blake-snc
force-pushed
the
fix/blackwell-error-messages-sm12x
branch
from
a5ac44c to
d03ac15
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
server_args.pyreference only "SM100" but the code usesis_blackwell_supported()which covers both SM100 and SM12x (consumer Blackwell, DGX Spark)Changes
...on SM100....on Blackwell.Detected SM100 and MXFP4...Detected Blackwell and MXFP4......Blackwell GPUs (SM100)......Blackwell GPUs (SM100/SM12x)...Context
is_blackwell_supported()incommon.pychecksdevice_capability_majors=[10, 12], so it returnsTruefor both SM100 (datacenter Blackwell) and SM12x (consumer Blackwell / DGX Spark). The log and error messages should reflect this.No behavior changes — only string literals updated.
Test plan
Identified on DGX Spark (SM121a) hardware provided by Second Nature Computing, an applied AI lab.
🤖 Generated with Claude Code