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

Gh032 myriad config #41

Merged
merged 21 commits into from
Feb 7, 2023
Merged

Gh032 myriad config #41

merged 21 commits into from
Feb 7, 2023

Conversation

Rachel-Alcraft
Copy link
Collaborator

closes #32

@Rachel-Alcraft Rachel-Alcraft added the enhancement New feature or request label Feb 6, 2023
@Rachel-Alcraft Rachel-Alcraft self-assigned this Feb 6, 2023
source ~/.bashrc
module load python3
module load java/openjdk-11/11.0.1
pip install --upgrade pip wheel pytest nextflow
Copy link
Collaborator

Choose a reason for hiding this comment

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

here the pip nextflow module installs a command "nextflow" into the venv which, when run, automatically installs nextflow for you and replaces itself with actual nextflow, and also ensures nextflow in the path, therefore if you install this pip module you shouldn't need to also curl install nextflow as well

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes thanks, my oversight, I'll remove the pip install.

pytest
cath-af-cli
# RUN NEXTFLOW
curl -s https://get.nextflow.io | bash
Copy link
Collaborator

Choose a reason for hiding this comment

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

see above comment: either use pip or curl install method for nextflow not both?

Copy link
Collaborator

@robertvi robertvi left a comment

Choose a reason for hiding this comment

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

Looks good.

@Rachel-Alcraft Rachel-Alcraft merged commit 81ee0f2 into main Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Myriad config script
2 participants