Skip to content
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

macOS specific? tweaks/performance improvements. #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

XPGeek
Copy link

@XPGeek XPGeek commented Jan 11, 2024

Good morning, I come from your YouTube channel here:
https://www.youtube.com/watch?v=CUjO8b6_ZuM

Running your tutorial (excellent job!) on my:

16-inch, 2021, M1 Max, 64GB, macOS 14.3 beta

I am sorry for beta configuration, I like testing macOS!

Changes

  • Alphabetical order of packages, makes it easier to scan?

  • Specified specific llama-cpp-python==0.1.48 due to dependency error on pip install -r requirements.txt

See: zylon-ai/private-gpt#113

  • Added watchdog package due to message on app start:

    For better performance, install the Watchdog module:

    $ xcode-select --install $ pip install watchdog

Love your content, thank you for your tutorial! Have you seen PrivateGPT?

https://github.com/imartinez/privateGPT

It can ingest PDF and more formats with local processing too, OpenAI API compatible too! Cheers, -Sam

- Specified specific llama-cpp-python==0.1.48 due to dependency error on pip install -r requirements.txt

- Added watchdog package due to message on app start:

  For better performance, install the Watchdog module:

  $ xcode-select --install
  $ pip install watchdog
@Leon-Sander
Copy link
Owner

Thanks, I like the reordering and appreciate the contribution.
I also added requirements_with_versions.txt and pip_freeze.txt which entails everything from my environment. Could you try those requirements to see if the dependency issue persists?

I am also going to try it out on my mac as well the next days. I didn't see the watchdog message on my linux system, so maybe comment watchdog out and people with macOS can remove the comment sign and install it.

Yeah I have seen PrivateGPT, pretty nice and I love the community contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants