Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

qwen2-vl infer report 'RuntimeError: Key and Value must have the same sequence length' #6348

Closed
1 task done
sun11 opened this issue Dec 16, 2024 · 0 comments · Fixed by #6359
Closed
1 task done
Labels
solved This problem has been already solved

Comments

@sun11
Copy link

sun11 commented Dec 16, 2024

Reminder

  • I have read the README and searched the existing issues.

System Info

  • llamafactory version: 0.9.2.dev0
  • Platform: Linux-6.5.0-26-generic-x86_64-with-glibc2.35
  • Python version: 3.10.15
  • PyTorch version: 2.2.0+cu118 (GPU)
  • Transformers version: 4.46.3
  • Datasets version: 3.1.0
  • Accelerate version: 1.0.1
  • PEFT version: 0.12.0
  • TRL version: 0.9.6
  • GPU type: NVIDIA GeForce RTX 4090
  • DeepSpeed version: 0.15.4
  • Bitsandbytes version: 0.44.1

Reproduction

FORCE_TORCHRUN=1 llamafactory-cli train examples/predict.yaml
训练qwen2-vl-7b模型一切正常,推断时报错:
'RuntimeError: Key and Value must have the same sequence length'

回退到commit:bcb4fb353ed810477b7813c6904b33c51efb7f81后推断qwen2-vl-7b正常,即fix mrope修改前

Expected behavior

No response

Others

No response

@github-actions github-actions bot added the pending This problem is yet to be addressed label Dec 16, 2024
hiyouga added a commit that referenced this issue Dec 17, 2024
@hiyouga hiyouga mentioned this issue Dec 17, 2024
2 tasks
@hiyouga hiyouga added solved This problem has been already solved and removed pending This problem is yet to be addressed labels Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved This problem has been already solved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants