-
Notifications
You must be signed in to change notification settings - Fork 38
Add developer conda quickstart #44
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
Add developer conda quickstart #44
Conversation
|
||
.. code-block:: console | ||
|
||
(base) C:\Users\wkheisenberg> mkdir labscript-suite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This folder is not consistent with the other documentation that uses C:\labscript-suite
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might be anticipating a prospective change being discussed in #43, whereby we recommend using the profile directory for a developer installation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I thought that was just going to be for the .venv, not the editable installs too. Did I misinterpret?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understood it to mean for both, which I think is a good idea.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A subdirectory of the profile directory would also be fine, but a different but inevitably similarly-named directory outside it is unappealing to me.
d3e8487
to
76790ed
Compare
Merging, and will complete the rest of the instructions (or make other changes) in subsequent PR(s) |
Currently not working because I need to:
setuptools-conda
into the labscript-suite repositorydesktop-app install
claiming e.g.blacs
isn't installed.Still coming: details section after quickstart