Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

Workaround for issue #2 #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

visze
Copy link

@visze visze commented May 27, 2015

Hotfix of the Ti/Tv ratio Bug. I implemented two checks.

  1. is refHom => Skip
  2. VariantAnnotation is compatible with Genotype.

The checks I implemented are not needed if the ordering of the functions
and commands are used in a clever way.

I would say:

  1. Iterating over the samples.
  2. Get Genotype and check it (homRef or noCall).
  3. Generate VariantAnnotations and iterate over them.

The check I implemented is not needed if the ordering of the functions
and commands are used in a clever way.

I would say: 
1) Iterating over the samples.
2) Get Genotype and check it (homRef or noCall).
3) Generate VariantAnnotations and iterate over them.
if (gt.isNoCall()) {
this.n_missing++;
return;
}
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to put this upwards

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

Successfully merging this pull request may close these issues.

1 participant