Commit 75b31bb
authored
Fix smollm2 alias to point at SmolLM2-135M (v2) instead of SmolLM-135M (v1) (#18859)
The original SmolLM2 PR (#9354) started as v1 support, was renamed to
`smollm2` during review, but the repo ID and `rope_theta` were never
updated to v2 values. The two checkpoints are genuinely different models
(0/272 tensors match).
- `HUGGING_FACE_REPO_IDS["smollm2"]`: `HuggingFaceTB/SmolLM-135M` →
`HuggingFaceTB/SmolLM2-135M`
- `examples/models/smollm2/135M_config.json`: `rope_theta` `10000.0` →
`100000.0` (matches [SmolLM2-135M HF
config](https://huggingface.co/HuggingFaceTB/SmolLM2-135M/blob/main/config.json))
### Test plan
Data-only change (one string, one number). Verified values match the
upstream HuggingFace SmolLM2-135M config.1 parent edb8c98 commit 75b31bb
2 files changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
126 | | - | |
| 126 | + | |
127 | 127 | | |
128 | 128 | | |
129 | 129 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
0 commit comments