-
Notifications
You must be signed in to change notification settings - Fork 20
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
Issue with the SortmeRNA process #141
Comments
@phuongdoand thanks for your interest into our pipeline and for reporting that issue. I think we (and others) experienced this now a couple of times. May I ask what your running We already thought about replacing the current SortMeRNA version with the newer one but had other issues then. Actually, we currently even think about replacing SortMeRNA w/ www.github.com/hoelzer/clean (but using the same database SortMeRNA provides). @MarieLataretu in the meantime it might be worth giving the newest SortMeRNA version another try? @phuongdoand sorry for the inconvenience, but in the case your RNA-Seq samples anyway have only a low rRNA amount (see the logs in the SortMeRNA working dirs) you might just want to skip the process via the respective flag (see (ps: Marie and I are on holiday so we might be a bit slower in responding/looking into this) |
Here we had some long discussion about this (and other) issues: #116 The user was running the pipeline w/ tmux and apparently this was causing strange issues with SortMeRNA in Nextflow that we were not able to fully unravel. So I bet that you also started the pipeline in a screen/tmux/... ? :) If so, can you please give it a try without? Maybe you are able to use Nextflows build-in |
@hoelzer Thank you very much for the prompt response and so sorry to disrupt you on your holiday. Forgot to attach my command in the issue but here it is:
I was indeed running the command with a screen session using the screen command. So is it causing sortmeRNA to not work properly? |
@phuongdoand no problem ;) Thanks for the command. btw it's always good to use the
And yes, we dont have an explanation yet but we also experienced this behavior of SortMeRNA in a screen session. It could be also that this only happens in the specific combination of SortMeRNA+Nextflow+Screen/tmux. It would be cool if you can test the same command w/o using a screen or the Another hint, w/
and then just
to check what's going on. |
@hoelzer Thank you for the tip and advice. Nevertheless, I just finished running the command again with nextflow So in the end, I decided to use the |
Okay, can you do a last test and just run the command without any
screen/bg? So simply in the terminal?
It would be interesting if this works then finally and sortmerna produces a
correct fastq.
If not, we really need to replace the tool.
…On Thu, 19 Aug 2021, 03:58 phuongdoand, ***@***.***> wrote:
@hoelzer <https://github.com/hoelzer> Thank you for the tip and advice.
Nevertheless, I just finished running the command again with nextflow -bg
flag but the same behavior occurred again. So I guess that this is not
because of using screen/tmux problem but rather sortmeRNA's trouble.
So in the end, I decided to use the --skip_sortmerna to avoid the error
and also to accelerate the analysis pipeline.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#141 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADN2CZ26LDYAOWI7QUEYNYLT5RQMFANCNFSM5CG3X4MA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
|
@hoelzer Sure, it will take quite some time for the sortmeRNA to run, so I will let you know the testing result when it is available. |
@hoelzer Got the result today, the problem still occurs on the FASTQ file that I have been using. Btw, since you mentioned the README, it would be better if you can add the headline in the |
@phuongdoand okay thanks for reporting! And weird - I thought that running the pipeline and thus SMR outside of a screen/bg might solve the issue. But then we really need to replace SMR w/ the newer version or another tool. I will make a separate issue for that. Regarding the README: thanks for the hint, this we can do! |
Hi, I just found out about your pipeline a few days ago and decided to give it a test run.
I tried it on two datasets that I am currently analyzing and in one of the datasets, there is an error that occurred in the
preprocess:hisat2
process.The error said that the length of the sequence and the length of the quality value are not the same, so I traced back to the fastq file and found the sequence that triggered the error. In the end, I found out that there are multiple sequences with the same behavior:
Just for reference, I also went back to check the original FastQ file provided by the sequencing machine and their sequence length is the same as the quality length:
Because the
preprocess:hisat2
used the*.other.fastq.gz
file (which is the output of sortmeRNA), I believe that the error actually occurred with thesortmeRNA
process when it merges the R1 and R2 to map to the rRNA genome then unmerge to returnother.R1
andother.R2
files.Since it is an old version of sortmeRNA and now they have fixed it, can you try to update sortmeRNA to a newer version?
The text was updated successfully, but these errors were encountered: