-
Notifications
You must be signed in to change notification settings - Fork 10
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
Conda documentation #408
Conda documentation #408
Conversation
Note that with conda installation you can't invoke the pipelines directly via Nextflow, so you will need to use the corresponding `cmat` commands - e.g. `cmat annotate` instead of `nextflow run annotation_pipeline.nf`. | ||
All the same command line options apply. |
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.
Should we make this our main mode of executing CMAT ?
It sounds more elegant and easier to document.
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.
Assuming you mean just here in the readme, yes probably, I was just being lazy... I'll update.
I think it would also work for our OT submissions, I wanted to try getting the commands to work outside of the conda install but we could honestly even just use the conda version on our cluster...
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.
Is there somewhere the full list of corresponding cmat
commands to use with conda?
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've created #409 for generalising the wrapper script (and hence the short cmat
commands), for now as you've suggested I've just included the corresponding cmat
commands in the readme.
- biopython ==1.77 | ||
- coverage ==6.5.0 | ||
- coveralls ==3.3.1 | ||
- jsonschema ==3.2.0 | ||
- numpy ==1.24.3 | ||
- pandas ==1.5.3 | ||
- pytest ==7.2.2 | ||
- pytest-cov ==2.10.0 | ||
- requests ==2.31.0 | ||
- requests-mock ==1.8.0 | ||
- retry ==0.9.2 |
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.
Is this some formatting standard I do not know about or an error ?
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.
It's a formatting standard that I also did not know about, bioconda utils lint picked it up in the PR.
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.
LGTM just a small comment
Note that with conda installation you can't invoke the pipelines directly via Nextflow, so you will need to use the corresponding `cmat` commands - e.g. `cmat annotate` instead of `nextflow run annotation_pipeline.nf`. | ||
All the same command line options apply. |
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.
Is there somewhere the full list of corresponding cmat
commands to use with conda?
No description provided.