Skip to content

Getting started: Use a venv instead of user-wide installation#8

Merged
gertjanvanzwieten merged 1 commit into
evalf:mainfrom
MakisH:venv
Jul 8, 2025
Merged

Getting started: Use a venv instead of user-wide installation#8
gertjanvanzwieten merged 1 commit into
evalf:mainfrom
MakisH:venv

Conversation

@MakisH
Copy link
Copy Markdown
Contributor

@MakisH MakisH commented Jul 8, 2025

The pip install --user is forbidden by default in some systems, such as on Ubuntu 24.04.
Looking at the documentation, the only place where this is mentioned is in the Getting Started page.

This PR changes the text to guide into using a venv instead.

Comment thread src/start.md
python -m pip install nutils matplotlib
```

Note that Nutils depends on Numpy, Treelog and Stringly, which means that these
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also delete this line, I think it does not add value (every Python package depends on other packages) and it is already outdated (at least Appdir is missing).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fully agree with losing --user, but the proposed line with source is linux specific. Instead I would just prefix the snippet with 'In a terminal window with an active virtual environment:'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, good point. My bad for thinking too much in Linux-only terms. Fixed.

@gertjanvanzwieten gertjanvanzwieten merged commit 6b670a6 into evalf:main Jul 8, 2025
1 check passed
@gertjanvanzwieten
Copy link
Copy Markdown
Contributor

Thanks!

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