transformers
Only default `rope_parameters` to empty `dict` if there is something to put in it
#42651
Merged

Only default `rope_parameters` to empty `dict` if there is something to put in it #42651

hmellor
hmellor Only default `rope_parameters` to empty `dict` if there is something …
fc4db6ed
HuggingFaceDocBuilderDev
hmellor hmellor requested a review from zucchini-nlp zucchini-nlp 10 days ago
zucchini-nlp
zucchini-nlp commented on 2025-12-05
hmellor Add warning
8012d222
zucchini-nlp
zucchini-nlp commented on 2025-12-05
hmellor Also catch explicit `{}`
10bfdf5f
zucchini-nlp
zucchini-nlp approved these changes on 2025-12-05
hmellor Merge branch 'main' into standardise-rope-fix
b70b8913
hmellor hmellor enabled auto-merge (squash) 9 days ago
hmellor Handle model with layer types and rope theta but not rope parameters
55d77f73
hmellor Merge branch 'main' into standardise-rope-fix
d5867570
github-actions
ArthurZucker
ArthurZucker approved these changes on 2025-12-10
disabled auto-merge 5 days ago
Manually disabled by user
ArthurZucker ArthurZucker merged 3230fb50 into main 5 days ago
hmellor hmellor deleted the standardise-rope-fix branch 5 days ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone