Skip to content
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

Update install instructions #8

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

carissableker
Copy link

Hi,

I've made a few small changes to allow installing the dependencies with conda easier.

The changes are:

  • Update the conda environment file.
    The existing file included many OS dependent libraries and pinned build versions. This makes it very specific to the platform the file was created on. I removed these, and only include the explicit dependencies, so that it can work on different architecture/OS/platforms (cf https://stackoverflow.com/a/39299669/4996681)
  • Added missing python library dependencies to the README
  • Updated the install instructions (including the use of conda to install the dependencies and the git clone command)
  • Changed how the phasiHunter bash script identifies the phasiHunter bin directory, so that is also works if phasiHunter is symlinked instead of being added to the $PATH

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.

1 participant