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

Pipeline won't run when cdc.config file missing #5

Closed
jessicarowell opened this issue Dec 8, 2023 · 3 comments
Closed

Pipeline won't run when cdc.config file missing #5

jessicarowell opened this issue Dec 8, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@jessicarowell
Copy link
Collaborator

Description of the bug

Line 184 in nextflow.config causes pipeline to fail because this file has not been included in the conf folder.

line 184: includeConfig 'conf/cdc.config'

We can remove this line, but then CDC users need to point to the cdc config file when they run the pipeline on Rosalind, and so we need to update README docs to indicate that. Otherwise, we include that file in conf. I can do whichever is preferred.

Command used and terminal output

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 -profile singularity,rosalind

File not found: /scicomp/home-pure/ick4/01.scripts/polkapox/conf/cdc.config

Relevant files

N/A

System information

No response

@jessicarowell jessicarowell added the bug Something isn't working label Dec 8, 2023
@jessicarowell jessicarowell self-assigned this Dec 8, 2023
@mikeyweigand
Copy link
Collaborator

Suggest to remove includeConfig 'conf/cdc.config from nextflow.config. Readme already has some basic instruction for CDC users to add their own config:
https://github.com/CDCgov/polkapox?tab=readme-ov-file#running-on-cdc-cluster

Suggest to update this with generic example of how to also define config at runtime, eg:
-config /some/path/to/cdc.config

@jessicarowell
Copy link
Collaborator Author

Done. Waiting to be able to test (missing mpxvgfa.sif)

@jessicarowell
Copy link
Collaborator Author

Merged into master by @mikeyweigand

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants