transformers
Only default `rope_parameters` to empty `dict` if there is something to put in it
#42651
Merged
Go
Login via GitHub
Home
Pricing
FAQ
Install
Login
via GitHub
Overview
Commits
6
Changes
View On
GitHub
Only default `rope_parameters` to empty `dict` if there is something to put in it
#42651
ArthurZucker
merged 6 commits into
huggingface:main
from
hmellor:standardise-rope-fix
Only default `rope_parameters` to empty `dict` if there is something …
fc4db6ed
hmellor
requested a review
from
zucchini-nlp
10 days ago
zucchini-nlp
commented on 2025-12-05
Add warning
8012d222
zucchini-nlp
commented on 2025-12-05
Also catch explicit `{}`
10bfdf5f
zucchini-nlp
approved these changes on 2025-12-05
Merge branch 'main' into standardise-rope-fix
b70b8913
hmellor
enabled auto-merge (squash)
9 days ago
Handle model with layer types and rope theta but not rope parameters
55d77f73
Merge branch 'main' into standardise-rope-fix
d5867570
ArthurZucker
approved these changes on 2025-12-10
disabled auto-merge
5 days ago
Manually disabled by user
ArthurZucker
merged
3230fb50
into main
5 days ago
hmellor
deleted the standardise-rope-fix branch
5 days ago
Login to write a write a comment.
Login via GitHub
Reviewers
ArthurZucker
zucchini-nlp
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Login to write a write a comment.
Login via GitHub