-
Notifications
You must be signed in to change notification settings - Fork 41
feat(qwen): add s1-mini-fp16 manifest and validation config #1131
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
da25cb3
6ee239f
507e223
42b4959
9fcb33b
33fd810
2ce4ebe
f0b440b
ca2425d
fd6a9cd
33bb4e3
09458c4
199742b
812e153
93ecd9f
e58a750
5ead3a8
6fc39bf
57b8362
253ca53
3e13fe1
b6067ea
b51c4ad
64bd3a9
3787692
c39f2da
9aeecae
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 |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| { | ||
| "name": "s1-mini-fp16", | ||
| "hf_id": "superwhisper/s1-mini", | ||
| "hf_revision": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", | ||
| "bundle": "s1-mini-fp16.bundle", | ||
| "family": "qwen", | ||
| "runtime_strategy": "qwen_decoder_kv_cache", | ||
| "task_strategy": "text_generation_causal", | ||
| "precision": "fp16", | ||
| "trust_remote_code": false, | ||
| "max_cache_length": 256, | ||
| "testcases": [ | ||
| { | ||
| "name": "s1-mini-fp16", | ||
| "reference_family": "chat_qwen3_posttrained", | ||
| "user_contract": "chat_response", | ||
| "reference_precision": "fp32", | ||
| "prompt": "[Styling: semi-formal] [Structure: prose] [Context: general] so um yeah for the trip we need to pack like sunscreen and uh chargers", | ||
| "max_new_tokens": 64, | ||
| "core": true | ||
| } | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| { | ||
| "threshold_overrides": { | ||
| "layer_atol": 5.0, | ||
| "logit_atol": 10.0, | ||
| "logit_cosine_p5": 0.0, | ||
| "logit_rel_l2_p95": 0.05, | ||
| "normalized_text_edit_distance": 1.0, | ||
| "stable_margin": 0.1, | ||
| "stable_top1_match_rate": 0.0, | ||
| "token_agreement_rate": 0.0, | ||
| "unstable_topk_hit_rate": 0.0 | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -3,6 +3,15 @@ | |||||||||
| "snapshot_date": "2026-08-23", | ||||||||||
| "purpose": "Documentation metadata only; checkpoint qualification remains defined by the README release row and E2E manifest.", | ||||||||||
| "checkpoints": [ | ||||||||||
| { | ||||||||||
| "hf_id": "superwhisper/s1-mini", | ||||||||||
| "revision": "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa", | ||||||||||
| "revision_source": "declared", | ||||||||||
| "metadata_file": "config.json", | ||||||||||
| "model_type": "qwen2", | ||||||||||
| "architectures": ["Qwen2ForCausalLM"], | ||||||||||
|
Comment on lines
+11
to
+12
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. 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win 🔎 Supported by static analysis🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- applicable repository guidance ---'
find /tmp/coderabbit-repo-knowledge/nvidia-tensorrt-model-connect-dd446d04 -type f -name '*.md' -maxdepth 3 -print
printf '%s\n' '--- metadata file ---'
cat -n website/data/hf-model-metadata.json
printf '%s\n' '--- references to metadata fields and architecture_source ---'
rg -n -C 3 'hf-model-metadata|architecture_source|model_type|architectures' website --glob '!**/node_modules/**'Repository: NVIDIA/TensorRT-Model-Connect Length of output: 50386 🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- repository-wide and website guidance ---'
cat /tmp/coderabbit-repo-knowledge/nvidia-tensorrt-model-connect-dd446d04/conventions/repo-wide.md
printf '%s\n' '--- architecture guidance for Qwen metadata ---'
cat /tmp/coderabbit-repo-knowledge/nvidia-tensorrt-model-connect-dd446d04/architecture/qwen3-8.md
printf '%s\n' '--- metadata consumers and local contract references ---'
rg -n -m 80 'hf-model-metadata|architecture_source|revision_source|metadata_file' . \
--glob '!website/data/hf-model-metadata.json' \
--glob '!**/node_modules/**'
printf '%s\n' '--- upstream config at the cited revision ---'
curl -fsSL 'https://huggingface.co/superwhisper/s1-mini/resolve/5ce55038d29cb7dac3031d4ef7fffe67bb9cd29e/config.json' \
| python3 -c 'import json,sys; d=json.load(sys.stdin); print(json.dumps({k:d.get(k) for k in ("model_type","architectures")}, indent=2))'Repository: NVIDIA/TensorRT-Model-Connect Length of output: 7228 🏁 Script executed: #!/bin/bash
set -e
printf '%s\n' '--- metadata validation and rendering ---'
sed -n '160,245p' website/plugins/model-support-inventory/index.js
printf '%s\n' '--- website documentation contract ---'
sed -n '1,45p' website/docs/models-recipes/overview.md
printf '%s\n' '--- relevant metadata test fixture ---'
sed -n '50,100p' website/plugins/model-support-inventory/index.test.jsRepository: NVIDIA/TensorRT-Model-Connect Length of output: 7084 Record the checkpoint’s Qwen3 architecture. The checkpoint Proposed fix- "model_type": "qwen2",
- "architectures": ["Qwen2ForCausalLM"],
+ "model_type": "qwen3",
+ "architectures": ["Qwen3ForCausalLM"],📝 Committable suggestion
Suggested change
🤖 Prompt for AI AgentsSource: MCP tools |
||||||||||
| "architecture_source": "config.architectures" | ||||||||||
| }, | ||||||||||
| { | ||||||||||
| "hf_id": "BAAI/bge-small-en-v1.5", | ||||||||||
| "revision": "5c38ec7c405ec4b44b94cc5a9bb96e735b38267a", | ||||||||||
|
|
@@ -14,6 +23,7 @@ | |||||||||
| ], | ||||||||||
| "architecture_source": "config.architectures" | ||||||||||
| }, | ||||||||||
|
|
||||||||||
| { | ||||||||||
| "hf_id": "Efficient-Large-Model/SANA-WM_bidirectional", | ||||||||||
| "revision": "e96271d77398def8ebb9fc595e7c0056dc625ab7", | ||||||||||
|
|
||||||||||
Uh oh!
There was an error while loading. Please reload this page.