-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Molmo-7B-D 4bit suddenly stopped working #160
Comments
temp.txt (BTW, same issue with python3.13) |
Oh, there was this change in the hugging-face 3 hours ago - https://huggingface.co/mlx-community/Molmo-7B-D-0924-4bit/commit/826ce5935256778523bfee774845f8387a2d6e3e And exactly those keys are missing. |
My bad! I'm refactoring utile and because of it I had change the model configs. I will make a new release fixing it in a few. |
Bear with me :) |
You can use this branch for now |
I tried the branch but it has the same issue of missing keys. No issues, I will wait for the refactor to finish. |
Seems the keys were removed a month ago, not 3 hours ago. 3 hours ago there was a minor change not related to keys: So, I am confused now what caused the issue then. I have tried 0.1.4 and 0.1.6 (and the refactor utils branch) |
It's fixed ✅ Please use the branch in this PR #161 |
What caused the issue is a change in the config file that is necessary for the upcoming changes. |
Works now. The released 0.1.6 itself worked (I havent tested the branch). |
Thanks! I found a better solution that is backwards compatible |
It was working yesterday (25/12/24) and now without any underlying change (I think) it starts complaining about missing params.
ValueError: Missing parameters: vision_tower.image_vit.patch_embedding.biases vision_tower.image_vit.patch_embedding.scales.
Full command -
(even without image, it gives same error)
Its a little weird, since neither mlx, nor mlx-vlm has changed yesterday (nor the weights from mlx-community on huggingface)
The text was updated successfully, but these errors were encountered: