-
Notifications
You must be signed in to change notification settings - Fork 187
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
Idefics3 Addition #379
Idefics3 Addition #379
Conversation
Sounds good to me, it's possible that there's nothing to change indeed. Because there are small discrepancies between generating with our internal codebase and Transformers integration, please ping me if the scores differ significantly from the officially reported ones |
@HugoLaurencon : Yes the changes are only to load the new model version of Idefics3 via config and re-use same style of inference as in Idefics2, other aspects remains constant. |
* VILA added * Update README.md * resolve config merge conflict * Fix error on Idefics for longer prompt * Fix naming convention to make consistent with Idefics2 and better readability * update config for idefics * Make LLava consistent as well * Add VILA 1.5 3B * Add VILA 1.5 3B * fix naming convention to be similar to the HF models * Multi-Turn added for Phi3-Vision and tested with MMDU * Add multi turn for Intern VL * fix formatting * Add Idefics3 Config * Warning message to build from source --------- Co-authored-by: aamita <aamita@sdg-slurm-bm-gpu-b4-8-ad3-009.compute.sdgdevvcn.oraclevcn.com> Co-authored-by: Junming Yang <[email protected]> Co-authored-by: Haodong Duan <[email protected]>
Idefics 3 follows same pattern as Idefics2.
Building HF from the Source Code and huggingface/transformers#32473 enables Idefics3
The model has been tested with the transformers library