Closed josruirod closed 2 years ago
Hi @josruirod,
Thanks for your words and for reporting this issue.
I will take a look at this ASAP. But at first glance, I think it might be related to an update in nextflow and how it is handling the integers and strings when read from command line. To guarantee that this may be the case, can you try passing this parameter through the samplesheet (as shown here), instead of the command line, please?
About the error with -c
, I really don't have a clue of what it may be. Can you share your config file and how you executed in that case?
For the problem related to the parameter, I created a new branch called params-issue
. Can you later try to run the pipeline using the code from this branch so we can check if it solves the issue before merging?
For that, you would need to attach the following in your command line: -r params-issue -latest
.
Hi, thanks for the prompt response! So few points to answer:
nextflow run fmalmeida/mpgap -c test.config
Best
Hi @josruirod,
I am not sure, but I think the problem on your config file is that you've set genome_size
param as a number:
genome_size = 23.3m
But this actually needs to be a string:
genome_size = "23.3m"
Not saying is really that, but is the only thing I could spot in a rapid look.
About the QC steps, actually they are performed in any strategy ... and is indeed super weird that your files are all blank. They should've at least have started and have something in it, even an error 🤔
Unfertunately, skippping is not possible, I may add in a future release, if you please open a new issue describing why you want it and flagging as "enchancement" 😄 .
But, even though you can't skip, I think is possible to ignore its errors and avoid pipeline crash if they fail, with a config file like this:
process {
withName: 'multiqc|quast' {
errorStrategy = 'ignore'
}
}
Please, let me know if that helps 😄
Thanks for your time and kind support!
So indeed, if I added genome size with quotes in the config file and used -c, it started but later complained also in relation with the hybrid strategy:
- --hybrid_strategy: '2' is not a valid choice (Available choices: 1, 2, both)
If I add quotes to hybrid strategy within the config, like hybrid_strategy = "2", then it works! But if I provide the parameter manually in the command line, --hybrid_strategy "2", same error.
About the QC, sure, I didn't mean that they are not necessary. I would like to get them to work. But maybe if they fail the whole pipeline shouldn't fail? Usable results could be obtained without requiring to restart the execution. I added those lines to ignore errors in the config file. It's currently running, will let you know if quast is bypassed. Maybe I'm opening another issue regarding with quast errors, so this one does not get too unrelated. Thanks
Thanks for that. I will continue to take a look. Just to make sure, you continued to observe the error with hybrid_strategy even with the new branch? Because when I use it, it does not complain about hybrid_strategy
anymore.
And I totally get what you mean about the QC. I think would be good to create a new issue to solve this quast error 😃
Hi, thanks for the work! So I'm happy to check, but can you please help? How can I run the new branch only? I've git cloned it, but the nextflow run command and some modifications I've tried seem to load the master branch only. Thanks
No, you don't need to clone it. I wrote here but you probably did not see 😄
You can use it directly from nextflow:
nextflow run fmalmeida/mpgap -r params-issue -latest [the rest of params you were using]
.
with -r
you tell nextflow which revision you want, either a branch or a release tag.
Oh my, apparently we wrote the mesage at exactly the same time, according to github, 20:54 CEST yesterday. What a sync. That's why I did not see your message. Sorry about that. So I can confirm with the params-issue what I observed with the hybrid seems to be solved. It does not complain like before and starts to run. Thanks for the help, let's see if I can make the whole pipeline to run
Hi, thanks for the impressive work and pipeline
So I'm trying to use it on our data, and getting a couple of errors for starters. If I ran the pipeline with --input XXXX.yml and --hybrid_strategy 2, I get the error:
both seems to be working with this syntaxis, but not 1 nor 2
If I ran the pipeline trying to provide the config file with -c, I get the error
I don't have much experience with nextflkow, so I may be missing something "easy". Hope you can comment and help. Thanks for the support