From 6c55658200db5870d032d52d04ce8746ee555607 Mon Sep 17 00:00:00 2001 From: Harry Mellor <19981378+hmellor@users.noreply.github.com> Date: Wed, 3 Dec 2025 12:51:59 +0100 Subject: [PATCH] Use `getattr` in `standardize_rope_params` because `rope_parameters` not always present Signed-off-by: Harry Mellor <19981378+hmellor@users.noreply.github.com> --- src/transformers/modeling_rope_utils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/transformers/modeling_rope_utils.py b/src/transformers/modeling_rope_utils.py index 542c81a4eaed..90a9b1f346bc 100644 --- a/src/transformers/modeling_rope_utils.py +++ b/src/transformers/modeling_rope_utils.py @@ -657,7 +657,7 @@ def standardize_rope_params(self): # Move `rope_theta` and `partial_rotary_factor` to the params dict, if not there yet rope_theta = getattr(self, "rope_theta", None) partial_rotary_factor = getattr(self, "partial_rotary_factor", None) - rope_parameters = self.rope_parameters or {} + rope_parameters = getattr(self, "rope_parameters", None) or {} # Case 1: RoPE param keys do not intersect with possible `layer_types` -> one global dict if getattr(self, "layer_types", None) is None or not set(rope_parameters.keys()).issubset(self.layer_types):