TinyLLaVA_Factory icon indicating copy to clipboard operation
TinyLLaVA_Factory copied to clipboard

Config for TinyLLaVA-OpenELM-270M-SigLIP-0.55B

Open eternalding opened this issue 1 year ago • 4 comments

Greetings. Would like to ask two questions about TinyLLaVA-0.55B (With OpenELM-270M-Instruct):

  1. From config.json provided in TinyLLaVA 0.55B's hf repo, seems it is using OpenELM-450M
  2. The repo title said it uses SigLIP for vision encoder, but config said it uses clip-vit-base-patch16.

Not sure if it's some kind of typo.

Thanks.

Link to TinyLLaVA 0.55B's config: https://huggingface.co/jiajunlong/TinyLLaVA-OpenELM-270M-SigLIP-0.55B/blob/main/config.json

eternalding avatar May 30 '24 06:05 eternalding

Thanks for your reminder, TinyLLaVA-0.55B actually uses OpenELM-450M-Instruct as the LLM and clip-vit-base-patch16 as the VisionTower. The config.json file in the Huggingface repository is correct. I have updated the description in the Huggingface repository. Thank you very much for pointing out the error.

jiajunlong avatar May 30 '24 11:05 jiajunlong

Okay. Thanks for the correction.

eternalding avatar May 31 '24 08:05 eternalding

Is there any TinyLLaVA version trained on OpenELM-270M-Instruct by any chance? @jiajunlong

ggcr avatar Jun 24 '24 10:06 ggcr

Is there any TinyLLaVA version trained on OpenELM-270M-Instruct by any chance? @jiajunlong

I'm so sorry, I just saw your message. We have tried using the OpenELM-270M-Instruct model, but the results were very poor.

jiajunlong avatar Jul 08 '24 08:07 jiajunlong