mskcc / facets

Algorithm to implement Fraction and Copy number Estimate from Tumor/normal Sequencing.
134 stars 65 forks source link

adding line in procSnps() to ensure required field order #194

Open agorelick opened 1 year ago

agorelick commented 1 year ago

In procSnps(), the rcmat data.frame must have columns in a specific order before its fields are renamed. My SNP matrix from snp-pileup did not match this order, causing depth data to be interpreted as vaf data, resulting in 0 het-SNPs being identified. I added a line to procSnps() to make sure the fields are in the correct order before they are renamed, so that the function can accept input data with arbitrary field order.