aldraus / quilt-llava

Codebase for Quilt-LLaVA
MIT License
34 stars 5 forks source link

Quilt-Llava is incompatible with up-to-date transformers versions #19

Open MoPl90 opened 5 months ago

MoPl90 commented 5 months ago

First of all: Thanks a lot for open-sourcing your work!

What this issue is about

While trying to reproduce some of the results mentioned in your paper, I noticed that the pinned version of the transformers package is quite outdated. Unfortunately, the code seems to be incompatible with up-to-date versions in its current state due to the AutoModel and AutoConfig classes encountering a name conflict with LLaVA, and some model-specific masking utilities for BLOOM and OPT.

Suggested changes

wisdomikezogwo commented 2 months ago

Thanks for the update, when I am back for the fall, I will test this and merge.