-
Notifications
You must be signed in to change notification settings - Fork 20
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
pip3 on macOS can install in funny directories #189
Conversation
Like: /Users/ecurtin/Library/Python//3.9/bin/omlmd Signed-off-by: Eric Curtin <ecurtin@redhat.com>
@tarilabs omlmd doesn't really work great for me at least on macOS, which is a pity, it's a very popular platform for model inferencing... $ ./ramalama.py pull oci://quay.io/mmortari/gguf-py-example:v1 |
That's weird because I made all the demos of omlmd on Mac OSX 👉 https://containers.github.io/omlmd/demos/demo/ |
Is the installation you are doing in a pyenv or venv? For another instance this is also analogously highlighted for ilab, here: https://github.com/instructlab/instructlab#:~:text=NOTE%3A%20The%20following%20steps%20in%20this%20document%20use%20Python%20venv%20for%20virtual%20environments. Happy to follow-up online to double check which steps you followed and potential more robust alternatives? |
Maybe it's just a harmless warning and https://quay.io/repository/mmortari/gguf-py-example isn't a real model |
if I recall I've taken that from the gguf example of llama.cpp repo and a valid gguf file but indeed not a Large Language Model |
I don't get those warning anyways on Mac OSX during my demo recordings, so I think it's a matter of how the library/dependencies are installed maybe? |
LGTM |
fwiw, just tried straight from repo root now and with:
I get just-and-only:
which then shows in my storage:
I believe the warn from #189 (comment) maybe depends on the local system environment and configuration used then? Possibly some my2c and happy to discuss further! |
...I think I see now; seems to me install.py is making use of the default pip3 from the default Mac global python environment: Lines 79 to 80 in 07555d5
what about:
please notice 2, 3 wouldn't however solve the same issue for the other dependency hence, I believe option 1 is the most preferable for the dependencies *of ramalama when installed/used on Mac. always happy to discuss further ! edit: typo |
@tarilabs if you could try and package ramalama and omlmd via brew that would be appreciated! We should migrate to there eventually... |
noted @ericcurtin eventually this will also impact however other dependencies of ramalama itself, as I noted in #189 (comment) |
Like:
/Users/ecurtin/Library/Python//3.9/bin/omlmd