-
Notifications
You must be signed in to change notification settings - Fork 27.6k
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 rope device matching bug #35505
Comments
It is related to OpenGVLab/InternVL#774 |
We are working on it! |
Hey @developer0hye, thanks for opening this issue! It appeared because we refactored the RoPE module to be inside the Model, instead of the Layers. Adding |
For BC we should probably just add that line no? |
#35583 will automatically take care of setting it if not provided EDIT: but it's best to explicitly set it, as custom device_map should be the responsibility of the user, this is only provided for BC |
@ArthurZucker Excecus me, what is BC? |
Backward compatibility 🤗 |
We decided internally not to automatically solve it in the end. I updated the snippet on the hub to avoid future issues for users though. For more details, see #35583! |
@Cyrilvallez I checked it! Thanks! |
System Info
transformers
version: 4.47.1Who can help?
I faced this bug running VLM InternVL2.5-78B.
text models: @ArthurZucker
vision models: @amyeroberts, @qubvel
Information
Tasks
examples
folder (such as GLUE/SQuAD, ...)Reproduction
Code sample
I tried to run demo code of
internVL2_5-78B
.Error
I printed device of its tensor and I got these results.
Expected behavior
No errors and print image caption.
The text was updated successfully, but these errors were encountered: