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

Issue7: update README #10

Merged
merged 3 commits into from
Dec 13, 2023
Merged

Issue7: update README #10

merged 3 commits into from
Dec 13, 2023

Conversation

jessicarowell
Copy link
Collaborator

First copy mpxvgfa.sif to assets.

Then test with:

nextflow run main.nf --indir /scicomp/groups-pure/Projects/scbs_mpob/JRowell_ick4/mpox_testing/paired/ --outdir /scicomp/groups-pure/Projects/scbs_mpob/JRowell_ick4/mpox_testing/results/ --project_name test -config /scicomp/reference/nextflow/configs/cdc-dev.config -profile singularity,rosalind

@jessicarowell jessicarowell linked an issue Dec 11, 2023 that may be closed by this pull request
@@ -159,17 +159,10 @@ For use on an HPC cluster (rosalind, aspen) the workflow can be run as a job by
module load nextflow
```

2) Prepare file `cdc.confing` with the necessary `sge` submission parameters, add it to sub-directory [./conf/](./conf), and include it in [nextflow.config](./nextflow.config) with:
Copy link
Collaborator

Choose a reason for hiding this comment

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

@jessicarowell couldn't this be either or? user could add their own cdc.conf with includeConfig OR just specify the scicomp reference cdc.conf.

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, it could be either-or. Just based on my conversations with scicomp, they'd rather people use the one they've preconfigured...and I think building one's own config file just adds unnecessary complexity. But I can add back that line if you wish!

@mikeyweigand mikeyweigand merged commit 529c4be into master Dec 13, 2023
1 of 9 checks passed
@kyleoconnell kyleoconnell deleted the issue7 branch February 5, 2024 15:37
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.

Update README.md
2 participants