docs(readme): recommend pipx for install (fixes PEP 668 + global dep conflicts)#1237
Open
OmkarKirpan wants to merge 1 commit intoMemPalace:developfrom
Open
docs(readme): recommend pipx for install (fixes PEP 668 + global dep conflicts)#1237OmkarKirpan wants to merge 1 commit intoMemPalace:developfrom
OmkarKirpan wants to merge 1 commit intoMemPalace:developfrom
Conversation
…flicts The current 'pip install mempalace' instruction either fails outright on PEP 668-managed Pythons (Debian/Ubuntu, Homebrew) or upgrades chromadb / numpy / grpcio / click in the user's global site-packages and breaks unrelated tools (numba, litellm, tutor, opentelemetry, ...). mempalace ships a CLI, so pipx (or 'uv tool install') is the right default — it isolates the install and still puts 'mempalace' on PATH. Plain pip is kept as the alternative for users who want 'import mempalace' inside their own venv. Refs MemPalace#284
23ff292 to
afa0c31
Compare
Member
|
Looks good and it's the way it should be installed indeed. @bensig @milla-jovovich ok on this ? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
pip install mempalaceline in the README either fails onPEP 668-managed Pythons (Debian/Ubuntu, Homebrew) or silently upgrades
chromadb/numpy/grpcio/clickin the user's globalsite-packages and breaks unrelated tools.
A real run on macOS today, against a Python that already had
numba,litellm,tutor, and the OpenTelemetry instrumentation packagesinstalled, produced this from a single
pip install mempalace:mempalace ships a CLI, so
pipx install mempalace(oruv tool install mempalace) is the right default — it isolates theinstall and still puts
mempalaceonPATH. Plainpip installiskept as the documented alternative for users who want
import mempalaceinside their own activated venv.Refs #284 (PEP 668 /
--break-system-packages).What changed
README.md"Install" section: lead withpipx install mempalace,mention
uv tool installas an equivalent, briefly explain why(PEP 668 + global dep conflicts), and keep
pip installfor thein-venv case.
Test plan
git diff README.md— only the Install section changespipx install mempalacesucceeds on a system Python wherepip install mempalacewould either trip PEP 668 ordowngrade/upgrade global packages