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

teststrandbias problem #83

Closed
srx3001 opened this issue Sep 7, 2018 · 2 comments
Closed

teststrandbias problem #83

srx3001 opened this issue Sep 7, 2018 · 2 comments

Comments

@srx3001
Copy link

srx3001 commented Sep 7, 2018

Hello,
I am testing VarDict on the sample files provided (my.bam and my.bed) and I always get an empty output.
The command I use is:

AF_THR="0.01"
vardict -G /mnt/Homo_sapiens.GRCh37.dna.primary_assembly.fa -f $AF_THR -N TP53 -b my.bam -c 1 -S 2 -E 3 -g 4 my.bed | teststrandbias.R | var2vcf_valid.pl -N TP53 -E -f $AF_THR

The error I get is:
Error: Incorrect input detected in teststrandbias.R
Execution halted

When I try to write the output to a file from vardict as so:

vardict -G /mnt/Homo_sapiens.GRCh37.dna.primary_assembly.fa -f $AF_THR -N TP53 -b my.bam -c 1 -S 2 -E 3 -g 4 my.bed > whatever.txt

I get a file with 36 columns and apparently teststrandbias expects 34 or 38. (ill upload it).
whatever.txt

How come this happens?
What are the steps I can take to solve this? Is it because your reference is different?

Thank you in advance,
Srdjan

@PolinaBevad
Copy link
Contributor

@srx3001, hello!
Sorry about this issue!
There are now 36 columns in the simple mode of Vardict (2 columns were added for SV analysis and duprate). We will fix the scripts soon, for now you can change condition in teststrandbias.R from (mynumcols == 34 || mynumcols == 38) to (mynumcols == 36 || mynumcols == 38). Columns for the amplicon mode are the same as before.

chapmanb added a commit that referenced this issue Sep 21, 2018
New structural variant addition to VarDictJava in 1.5.5 or later has 36
columns. Fixes #83
@dattam
Copy link

dattam commented Sep 26, 2018

@srx3001, hello!
Sorry about this issue!
There are now 36 columns in the simple mode of Vardict (2 columns were added for SV analysis and duprate). We will fix the scripts soon, for now you can change condition in teststrandbias.R from (mynumcols == 34 || mynumcols == 38) to (mynumcols == 36 || mynumcols == 38). Columns for the amplicon mode are the same as before.

@PolinaBevad , the R script uses the following colClasses, but they don't match the (36 column output). Also, I suppose this incompatibility will also create issues with var2vcf_valid.pl. Do you have any suggestions for a short term fix. Maybe I can "cut" specific columns and feed it to these scripts ? In that case, is there a way to identify the "older" list of columns in the new output ? Thanks!

colClasses=c("character", NA, NA, NA, NA, "character", "character", NA,
NA, NA, NA, NA, NA, "character", NA, NA, NA, NA, NA, NA, NA, NA).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants