Hey there about clip.cpp HOT 4 CLOSED

Thanks @Green-Sky for your attention and helpful feedback.

had to manually checkout your master.

Unfortunately, this is because ggerganov/ggml#254 has not been merged yet. I'll checkout upstream master once it's merged. Maybe I can request Georgi's attention in that PR.

i got a better model

Thanks for trying this. It was also in my mind to try Laion's models, and seems that your modification sould work for both cases. I'll test it and modify the script to handle models from both sources.

hardcoded paths in examples/main

Sorry for that, I'm a little bit lazy to pass them from command line every time I'm running during the development 😶‍🌫️ The next commit today will remove them and parse them from command line instead.

I'll also share examples for zero-shot image labelling (one image vs. multiple texts), and semantic image search (multiple images vs. one textual query).

from clip.cpp.

Got it. Fixed it now.
Also removed hardcoded stuff from the example and it reads from the command line. Run ./bin/main -h for options.

Later today or tomorrow, I'll update the conversion script to handle models from Laion and craft better examples.

Please feel free to continue sharing your feedback, highly appreciated.

from clip.cpp.

had to manually checkout your master.

Unfortunately, this is because ggerganov/ggml#254 has not been merged yet. I'll checkout upstream master once it's merged. Maybe I can request Georgi's attention in that PR.

What I meant is that the commit should be one in your fork (for the pr). But even github cant find the exact commit, which means you never pushed the committed commit.

from clip.cpp.

closing this as done :)

from clip.cpp.