Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged

Conversation

chrisjbillington
Copy link
Member

@chrisjbillington chrisjbillington commented Jun 6, 2020

Currently not working because I need to:

  • Get setuptools-conda into the labscript-suite repository
  • Fix a bug whereby setuptools-conda is not installing dependencies of all packages correctly - it seems to install the dependencies of one of them repeatedly instead.
  • fix a bug in desktop-app, it gives an error on desktop-app install claiming e.g. blacs isn't installed.

Still coming: details section after quickstart


.. code-block:: console

(base) C:\Users\wkheisenberg> mkdir labscript-suite
Copy link
Member

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

Copy link
Member

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.

Copy link
Member

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?

Copy link
Member Author

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.

Copy link
Member Author

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.

@chrisjbillington
Copy link
Member Author

Merging, and will complete the rest of the instructions (or make other changes) in subsequent PR(s)

@chrisjbillington chrisjbillington merged commit bc937a4 into labscript-suite:master Jun 9, 2020
@chrisjbillington chrisjbillington deleted the dev-conda-docs branch June 9, 2020 01:44
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.

3 participants