Document cmake option: NRN_INSTALL_PYTHON_PREFIX#3747
Document cmake option: NRN_INSTALL_PYTHON_PREFIX#3747ctrl-z-9000-times wants to merge 3 commits intoneuronsimulator:masterfrom
Conversation
Up to standards ✅🟢 Issues
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3747 +/- ##
==========================================
- Coverage 68.33% 68.32% -0.01%
==========================================
Files 691 691
Lines 111113 111113
==========================================
- Hits 75930 75920 -10
- Misses 35183 35193 +10 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
|
||
| NRN_INSTALL_PYTHON_PREFIX:STRING="lib/python/neuron/" | ||
| ----------------------------------------------------- | ||
| Path where NEURON Python components will be installed |
There was a problem hiding this comment.
Unless building a wheel (then the value will be forced to "neuron/")
|
|
||
| .. code-block:: shell | ||
|
|
||
| python -m site --user-site |
There was a problem hiding this comment.
It might be worth adding a note that the above doesn't resolve virtual environments, so one should install packages to this path with caution - I hope that many people use virtual environments now (through uv) to keep their dependencies separate, so using this facility might have surprising effects.
There was a problem hiding this comment.
venv installs it's packages to "venv_root/lib/python3.10/site-packages/" but I don't know how the user can find this automatically.
Instead I changed the documentation to say:
"Note: Non-standard installations (such as python virtual environments) have non-standard install paths."
|
3 participants
Fixes issue #3746