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

--n_haplotypes must be a required argument again #218

Closed
subwaystation opened this issue Jan 17, 2025 · 4 comments
Closed

--n_haplotypes must be a required argument again #218

subwaystation opened this issue Jan 17, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@subwaystation
Copy link
Collaborator

Description of the bug

With the new template update we miss this. I didn't spot this, because I always used the test profile for testing, but I did not test by hand.
I need to check what we need to add again @heuermh.

Command used and terminal output

Relevant files

No response

System information

No response

@subwaystation subwaystation added the bug Something isn't working label Jan 17, 2025
@subwaystation
Copy link
Collaborator Author

I spotted it now, because the resulting graph looks bad without specifying a reasonable value.

@subwaystation
Copy link
Collaborator Author

subwaystation commented Jan 17, 2025

It is getting weird. It's already set as required at

"required": ["input", "n_haplotypes", "outdir"],
.
However when I run

nextflow run main.nf -profile docker --outdir sun --input ../pggb/data/HLA/DRB1-3123.fa.gz -config conf/hla.config

There is no complaint that the parameter is missing. The pipeline runs through with it's default value 0 I assume.
Any ideas @heuermh?

This was referenced Jan 17, 2025
@subwaystation
Copy link
Collaborator Author

I need to set it to null apparently. See #220

@heuermh
Copy link
Contributor

heuermh commented Jan 17, 2025

Fixed by #219

@heuermh heuermh closed this as completed Jan 17, 2025
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

Successfully merging a pull request may close this issue.

2 participants