-
Notifications
You must be signed in to change notification settings - Fork 244
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
support Internvl chat llava #1426
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May update doc
Since a new vision model of latest llava is implemented, do we have to install internvl? |
We have to install internvl-managed llava |
verify the following cases with model InternVL-Chat-ViT-6B-Vicuna-7B
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested pipeline and api_server of InternVL-Chat-ViT-6B-Vicuna-7B
Motivation
support
Modification
The internvl-chat-llava developed upon older version of llava, so I write a new vision model. And there may be some bugs in their code OpenGVLab/InternVL#86