-
Notifications
You must be signed in to change notification settings - Fork 111
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
Metabat fails when running with multiple input files. #32
Comments
Hi! Thanks for reporting this. I'll take a look and come back to you. Best, |
I doubt it has anything to do with #27, because #27 is happening afterwards. Maybe bam sorting interferes? But as far as I understood, it doesn't sort the contigs, does it? @HadrienG did you test that already? |
It's the channel mixing that seems to combine the wrong samples together. I don't have a fix yet, but will look at it more today |
That's unfortunate that this slipped the QC before release. Maybe include @maxulysse into solving this since he added the channel magic? I actually also have four samples that I wanted to have analyzed in one go instead of individually. |
I'll have a look right away, thanks for mentioning me @d4straub |
So far I haven't been able to reproduce the bug. I just noticed @grst used It's possible that UPDATE: still not crashing for me with |
Thanks for the updates, I just started to run my samples without skipping anything, will see what happens. |
I got the same error as well, but I did not use |
@d4straub did you run into this as well? |
Sorry for the late answer. Yes I did run it and it worked fine. edit: we need a dataset / command to reproduce this issue to trouble shoot it. |
I still cannot reproduce this. Does anyone else had that problem? |
I did have that problem again, but with datasets different than the ones I commented about on March 4. |
Finally I also encountered that problem! Currently, I suspect that it is caused by parallelization and it seems to help to set the numbers of cores to 1 for that process. But my pipeline run isn't finished yet, will see if it indeed does help. |
Alright, I resumed the run that failed because of the described error with the additional parameter
and the run ended successful. |
PR is made against dev that should solve it, at least I hope so. |
Thank you @d4straub ! I will give this a try. |
This issue should be solved in the latest dev version. |
Thank you @skrakau! Will give this a try! |
Hi @skrakau. This is working when I run the latest dev version. Thank you! |
When I run the pipeline on multiple samples, the metabat2 step fails for me with the following error message:
The pipeline runs fine when I include only a single sample in the input directory. This might be related to #27.
Command executed
Input data
I ran the pipeline of publicly available metagenomics samples from the ibdmdb project. The fastq files can be downloaded here: https://ibdmdb.org/tunnel/public/HMP2/WGS/1818/rawfiles. For testing, I ran the pipeline on the 10 first samples listed in the web portal.
Full log
The text was updated successfully, but these errors were encountered: