-
Notifications
You must be signed in to change notification settings - Fork 6
Add model configs: GLM, DeepSeek V4, Inkling, Laguna, Laguna NVFP4 #39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -76,7 +76,7 @@ class RedHatAI_NVIDIA_Nemotron_3_Super_120B_A12B_NVFP4(ModelConfig): | |
| ] | ||
|
|
||
| class RedHatAI_Qwen3_6_27B_FP8(ModelConfig): | ||
|
|
||
| name = "RedHatAI/Qwen3.6-27B-FP8" | ||
| model_max_len = 131072 | ||
| args = [ | ||
|
|
@@ -90,3 +90,116 @@ class RedHatAI_Qwen3_6_27B_FP8(ModelConfig): | |
| "--tool-call-parser", "qwen3_coder", | ||
| "--default-chat-template-kwargs", '{"enable_thinking": true}', | ||
| ] | ||
|
|
||
|
|
||
| class RedHatAI_GLM_5_2_FP8(ModelConfig): | ||
| # Verified: 8x H200 141GB, concurrency 2.23x at 262K context | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think this is enough concurrency for us to consider using vLLM over OpenRouter for GLM 5.2. Let's keep it in the PR though for informational purposes. How does it do on 8x B200? |
||
| # Note: cannot fit 1M context on 8x H200 (needs 52.68 GiB KV, only 23.78 GiB available) | ||
|
|
||
| name = "RedHatAI/GLM-5.2-FP8" | ||
| model_max_len = 262144 | ||
| args = [ | ||
| "--model", "RedHatAI/GLM-5.2-FP8", | ||
| "--dtype", "auto", | ||
| "--max-model-len", "262144", | ||
| "--trust-remote-code", | ||
|
taagarwa-rh marked this conversation as resolved.
|
||
| "--kv-cache-dtype", "fp8", | ||
| "--enable-expert-parallel", | ||
| "--enable-auto-tool-choice", | ||
| "--reasoning-parser", "glm45", | ||
| "--tool-call-parser", "glm47", | ||
| ] | ||
|
|
||
|
|
||
| class RedHatAI_DeepSeek_V4_Flash(ModelConfig): | ||
| # Verified: 8x H200 141GB, concurrency 9.91x at 1M context | ||
| # Note: --moe-backend deep_gemm_mega_moe is B200-only (SM100) | ||
|
|
||
| name = "RedHatAI/DeepSeek-V4-Flash" | ||
| image = "vllm/vllm-openai:v0.27.1" | ||
| model_max_len = 1048576 | ||
| args = [ | ||
| "--model", "RedHatAI/DeepSeek-V4-Flash", | ||
| "--dtype", "auto", | ||
| "--trust-remote-code", | ||
| "--kv-cache-dtype", "fp8", | ||
| "--block-size", "256", | ||
| "--enable-expert-parallel", | ||
| "--enable-auto-tool-choice", | ||
| "--tokenizer-mode", "deepseek_v4", | ||
| "--tool-call-parser", "deepseek_v4", | ||
| "--reasoning-parser", "deepseek_v4", | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. https://recipes.vllm.ai/deepseek-ai/DeepSeek-V4-Flash mentions a couple flags I don't see here: Did you test these and find they are not needed?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh that is something I hadn't considered. Should we add a per-hardware args section to the ModelConfigs? E.g. Then somewhere when building the vLLM command it can reference Regarding
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. +1 on the hardware_extra_args approach. start_model already extracts the GPU preset from the instance so we can match on that. I'll implement it. DeepSeek gets |
||
| ] | ||
|
|
||
|
|
||
| class RedHatAI_DeepSeek_V4_Flash_NVFP4_FP8(ModelConfig): | ||
| # Verified: 8x H200 141GB, concurrency 9.81x at 1M context | ||
| # Note: Marlin FP4 fallback on H200 (no native SM100 FP4), similar concurrency to base | ||
|
|
||
| name = "RedHatAI/DeepSeek-V4-Flash-NVFP4-FP8" | ||
| image = "vllm/vllm-openai:v0.27.1" | ||
| model_max_len = 1048576 | ||
| args = [ | ||
| "--model", "RedHatAI/DeepSeek-V4-Flash-NVFP4-FP8", | ||
| "--dtype", "auto", | ||
| "--trust-remote-code", | ||
| "--kv-cache-dtype", "fp8", | ||
| "--block-size", "256", | ||
| "--enable-expert-parallel", | ||
| "--enable-auto-tool-choice", | ||
| "--tokenizer-mode", "deepseek_v4", | ||
| "--tool-call-parser", "deepseek_v4", | ||
| "--reasoning-parser", "deepseek_v4", | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See comment above |
||
| ] | ||
|
|
||
|
|
||
| class RedHatAI_Inkling_Small(ModelConfig): | ||
| # Verified: 8x H200 141GB, BF16, concurrency 13.22x at 1M context | ||
|
|
||
| name = "RedHatAI/Inkling-Small" | ||
| image = "vllm/vllm-openai:v0.27.1" | ||
| model_max_len = 1048576 | ||
| args = [ | ||
| "--model", "RedHatAI/Inkling-Small", | ||
| "--dtype", "auto", | ||
| "--trust-remote-code", | ||
| "--kv-cache-dtype", "fp8", | ||
| "--enable-auto-tool-choice", | ||
| "--tool-call-parser", "inkling", | ||
| "--reasoning-parser", "inkling", | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. https://recipes.vllm.ai/thinkingmachines/Inkling-Small?variant=bf16 has a couple flags I don't see here: Did you test these and find they are not needed? |
||
| ] | ||
|
|
||
|
|
||
| class RedHatAI_Laguna_S_2_1(ModelConfig): | ||
| # Verified: 8x H200 141GB, BF16, max-model-len 1048576, concurrency 30.61x | ||
|
|
||
| name = "RedHatAI/Laguna-S-2.1" | ||
| model_max_len = 1048576 | ||
| args = [ | ||
| "--model", "RedHatAI/Laguna-S-2.1", | ||
| "--dtype", "auto", | ||
| "--trust-remote-code", | ||
| "--kv-cache-dtype", "fp8", | ||
| "--enable-auto-tool-choice", | ||
| "--reasoning-parser", "poolside_v1", | ||
| "--tool-call-parser", "poolside_v1", | ||
| "--default-chat-template-kwargs", '{"enable_thinking": true}', | ||
| ] | ||
|
|
||
| class poolside_Laguna_S_2_1_NVFP4(ModelConfig): | ||
| # Verified: 1x B200 183GB, NVFP4, max-model-len 1048576, concurrency 2.33x | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you for testing this, same as above looks like this won't be enough concurrency so we'll prefer the BF16 version you added. No changes needed here. Thanks again for trying this, this will serve as justification for using the larger node to run this model |
||
|
|
||
| name = "poolside/Laguna-S-2.1-NVFP4" | ||
| model_max_len = 1048576 | ||
| args = [ | ||
| "--model", "poolside/Laguna-S-2.1-NVFP4", | ||
| "--dtype", "auto", | ||
| "--trust-remote-code", | ||
| "--kv-cache-dtype", "fp8", | ||
| "--enable-auto-tool-choice", | ||
| "--reasoning-parser", "poolside_v1", | ||
| "--tool-call-parser", "poolside_v1", | ||
| "--default-chat-template-kwargs", '{"enable_thinking": true}', | ||
| ] | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.