423A35C7/LLaMA-Factory
1
0
Fork 0
mirror of https://github.com/hiyouga/LLaMA-Factory.git synced 2026-08-17 13:35:44 +08:00
Code Issues Packages Projects Releases Wiki Activity

[data] pad position_ids on non-FA2 packing path (fixes rotary crash for Gemma-3/4) (#10737)

Browse Source 
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
haqishen
2026-08-09 17:00:10 +09:00
committed by GitHub
parent 887b850813
commit 63a89710c7
1 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/llamafactory/data/collator.py
View File

@@ -541,6 +541,17 @@ class SFTDataCollatorWith4DAttentionMask(MultiModalDataCollatorForSeq2Seq):
self._unpad_packed_features(features)
features["attention_mask"] = None # let transformers handle causal packed mask.
else:
# `DataCollatorForSeq2Seq(pad_to_multiple_of=...)` pads `input_ids`/`attention_mask`
# but leaves `position_ids` untouched (it is not in `model_input_names`). On the
# non-FA2 packing path we do not unpad, so `position_ids` stays shorter than
# `input_ids`, which makes cos/sin shorter than query and crashes
# `apply_rotary_pos_emb`. Right-pad `position_ids` to the padded length to match.
position_ids = features.get("position_ids")
if torch.is_tensor(position_ids):
pad_len = features["input_ids"].shape[-1] - position_ids.shape[-1]
if pad_len > 0:
features["position_ids"] = F.pad(position_ids, (0, pad_len), value=0)
for key, value in features.items(): # cast data dtype for paligemma
if torch.is_tensor(value) and torch.is_floating_point(value):