zzh
/
gatk-3.8
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Comments for Chris on cleanup

This commit is contained in:
Mark DePristo 2012-08-24 16:04:58 -04:00
parent 649b82ce85
commit 0996bbd548
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
public/java/src/org/broadinstitute/sting/gatk/walkers/annotator/MVLikelihoodRatio.java
View File

@ -71,6 +71,7 @@ public class MVLikelihoodRatio extends InfoFieldAnnotation implements Experiment
public List<VCFInfoHeaderLine> getDescriptions() { return Arrays.asList(new VCFInfoHeaderLine(MVLR_KEY, 1, VCFHeaderLineType.Float, "Mendelian violation likelihood ratio: L[MV] - L[No MV]")); }
// todo - this entire function should be in samples DB
private Set<Trio> checkAndSetSamples(SampleDB db){
Set<Trio> trioSet = new HashSet<Trio>();
for ( String familyString : db.getFamilyIDs() ) {
@ -97,6 +98,10 @@ public class MVLikelihoodRatio extends InfoFieldAnnotation implements Experiment
return true;
}
// TODO -- this class is too much.
// TODO -- Why iterable?
// TODO -- shuoldn't this be in samplesDB() so you can just called samplesDB().getTrios()
// TODO -- should just have final string IDs, and getters, no setters
private class Trio implements Iterable<String> {
private String maternalID;
private String paternalID;