Comments (9)
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
from mpgap.
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
from mpgap.
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
from mpgap.
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?
from mpgap.
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
.
from mpgap.
Hi, thanks for the prompt response! So few points to answer:
- Seems to be working if I include the hybrid strategy in the samplesheet
- About the error with -c, find attached the config file, and the execution was just:
test.config.txt
nextflow run fmalmeida/mpgap -c test.config
- Couple of comments. I got it working in the both hybrid strategy. Quast failed and weirdly enough, the command.sh, stderror, etc, in essence the files in the folder to debug were empty. I'm probably more interested in strategy 2 either way, and I think quast is in strategy 1, right? In any case, would the pipeline finish if I had let it run, the QC is not essential, right? Can Quast and MultiQC be skipped somehow, maybe similar to the assemblers?
Best
from mpgap.
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
from mpgap.
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
from mpgap.
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.
from mpgap.
Related Issues (20)
- Improve quality assessment HOT 3
- Check new assemblers? HOT 4
- Add option hifi
- Add an option for multiple samples HOT 9
- conda? HOT 11
- Shovill with all assemblers? HOT 2
- add polypolish tool HOT 1
- update documentation about the configuration in either config or samplesheet HOT 1
- change to unicycler v0.5.0? HOT 1
- new directory called "final_output"
- add testing git actions for PRs HOT 1
- use nf-core framework for CLI help and log messages HOT 3
- quast generating empty files HOT 24
- mkdir: cannot create directory HOT 4
- add trycycler
- add 3 hybrid strategy HOT 1
- Requesting support with error "Explicit 'name separator' in class" HOT 4
- update tool versions
- add homopolish tool HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mpgap.