Skip to content

Conversation

@mudler
Copy link
Owner

@mudler mudler commented Jan 2, 2026

Description

Construct the mmproj path from the golang side of things (which makes it more stable and predictable, and doesn't require backend rebuild)

Problem: when importing LocalAI puts mmproj files of llama.cpp in llama-cpp/mmproj, and models in llama-cpp/models. However, this doesn't work as expected because we assume that mmproj files are in the same paths of the model file.

This PR makes sure we join the path with the model path as expected (top-level).
Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@netlify
Copy link

netlify bot commented Jan 2, 2026

Deploy Preview for localai ready!

Name Link
🔨 Latest commit b9e3c20
🔍 Latest deploy log https://app.netlify.com/projects/localai/deploys/6957f3cdc5c7bb000831011b
😎 Deploy Preview https://deploy-preview-7832--localai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mudler mudler changed the title fix(mmproj): fix loading mmproj in nested sub-dirs fix(mmproj): fix loading mmproj in nested sub-dirs different from model path Jan 2, 2026
@mudler mudler added the bug Something isn't working label Jan 2, 2026
@mudler mudler changed the title fix(mmproj): fix loading mmproj in nested sub-dirs different from model path fix(llama.cpp/mmproj): fix loading mmproj in nested sub-dirs different from model path Jan 2, 2026
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
@mudler mudler merged commit 5f6c941 into master Jan 2, 2026
40 of 41 checks passed
@mudler mudler deleted the fix/mmproj branch January 2, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants