Commit Graph

  • c3c66e853c Improvements for Jason depristo 2010-07-09 20:18:37 +0000
  • 405be230d0 Various code improvements based on FindBugs ebanks 2010-07-09 15:04:48 +0000
  • abaec13e38 Bug fix: if there are samples in the VCF but all of them are no-calls, we still need to emit GT for the FORMAT field to be on spec. Note that this is a holdover from 3.3 writing but can't easily be fixed there. Fortunately, that code is all going away soon... ebanks 2010-07-09 14:08:25 +0000
  • ea8fd506bf Update to PickSequenomProbes: Option to ignore mask sites within X bp of a variant (very useful for indels where dbSNP entries near the indel are almost always false SNP calls). Also fixed an integration test where the variant site itself, being in dbSNP, was represented as [N/C] rather than [A/C]. Added integration test for 1bp no-mask window. chartl 2010-07-09 04:03:19 +0000
  • 179067e3f4 Support for . values in qual field depristo 2010-07-09 01:47:02 +0000
  • dd978dd525 misc. changes to python scripts depristo 2010-07-09 00:13:35 +0000
  • 6ffcaa0afe Can run R scripts on the command line depristo 2010-07-09 00:13:18 +0000
  • 45fb614296 Fixes to VE for obscure bug, as well as disabled integration test for CombineVariants depristo 2010-07-09 00:13:07 +0000
  • 67f1589652 --fdr_filter_level isn't mandatory rpoplin 2010-07-08 22:48:30 +0000
  • 5d39cd5db8 Added --fdr_filter_level to ApplyVariantCuts so that you can create beautiful tranche plots and also decide which tranche level to filter at. The previous version always filtered at the smallest tranche. The tranche filter names are appropriately added to the VCF header. rpoplin 2010-07-08 22:44:10 +0000
  • 760aaeda88 Update to CombineVariants. Now splits merge options into variant and genotype options separately. depristo 2010-07-08 20:09:48 +0000
  • bd2ba3eb37 deal with very large known indels that fall off our ref context ebanks 2010-07-08 20:05:16 +0000
  • 7ff6106c14 adding Ivy lines for findbug, and adding a build task (to run it locally you need to have installation of findbug). I'll put more information on the wiki when it's up and running. aaron 2010-07-08 19:10:19 +0000
  • 12fecc8d8f remove the picard DbSNP ROD. aaron 2010-07-08 17:46:00 +0000
  • c80a257f37 Updated for CentOS changes. See: http://iwww.broadinstitute.org/itsystems/wiki/Dotkit#Job_Specific_Wrapper_Script kshakir 2010-07-08 14:36:24 +0000
  • 56a0c7ee6f All headers are now converted to VCF4 by default. depristo 2010-07-08 14:14:17 +0000
  • 6e6ad36523 reallow MNP events through ebanks 2010-07-08 06:26:52 +0000
  • ed0d0d78fa corresponding fix for dealing with insertions ebanks 2010-07-08 05:25:03 +0000
  • ada8c9931f We were never clipping the VCF-provided ref base off the left end of the alleles for insertions, so the reference allele was never null (and downstream walkers would fail). Didn't this get tested with insertions at some point? ebanks 2010-07-08 05:24:27 +0000
  • 9a81f1d7ef Fixed this tool for chartl so that it now properly handles deletions. Added deletion case to integration tests. ebanks 2010-07-08 04:45:59 +0000
  • 47a42b1507 trivial cleanup ebanks 2010-07-08 04:42:32 +0000
  • b7a3d1e61f Bug fix: if the FORMAT field consisted of just GT, we were exceptioning out. How did we not catch this until now? ebanks 2010-07-08 04:41:40 +0000
  • 1c146aebe8 Fix logic bug ebanks 2010-07-08 04:32:46 +0000
  • 9fc05ac2ae eagerDecode is now false. hanna 2010-07-07 22:51:48 +0000
  • 4bc3ad2194 Shame on me: UG was emitting negative QUALs (-0) in all_bases mode. Thanks, Matt. ebanks 2010-07-07 20:30:22 +0000
  • 30714ec8d9 As per quick chat with Richard Durban, don't increase the mapping quality of realigned reads too much; for now, arbitrarily increase the MQ by 10. We need to figure out a better solution. ebanks 2010-07-07 20:12:59 +0000
  • 8ff1a4b929 Don't try to clean reads that fail the PF, in preparation for Ryan ebanks 2010-07-07 19:49:36 +0000
  • 6a0483ee95 fixed typos ebanks 2010-07-07 17:41:44 +0000
  • 2ac74a862d updated to deal with new cleaner options ebanks 2010-07-07 17:06:00 +0000
  • b934cc7554 Updates to fix some bugs in merger. Now able to merge into project wide indel VCF files. Integration teests coming tomorrow depristo 2010-07-07 03:16:33 +0000
  • 7be8c35eb2 Workaround for scala trait erasing parameterized types: - Requiring explicit @ClassType on parameterized fields in traits. - Scatter / Gather functions are now abstract classes since @ClassType can't be used on parameterized fields with type parameters. kshakir 2010-07-07 03:15:10 +0000
  • 120f90da5b Interval support for ref walkers while streaming. hanna 2010-07-07 03:14:59 +0000
  • 773a72e6ea An initial fix for performance issues when filtering UG with new StratifiedAlignmentContext. hanna 2010-07-07 01:07:46 +0000
  • be75b087ec a) Add input argument (-ncrate) to BeagleOutputToVCFWalker. If the genotype posterior error probability is higher than this threshold, we declare No-call at this genotype. b) Add "OG" annotation to genotypes. If Beagle changes genotypes, this annotation gets the original genotype call, to ease performance comparisons. If not, this annotation gets an empty value. delangel 2010-07-06 18:33:28 +0000
  • 4213e05aeb Fix for sharding ref walkers via monolithic sharding. Introduces the potential bug (for monolithic sharding only) that when traversing by read, map() function will not be called for loci off the end of the reference. hanna 2010-07-06 04:34:38 +0000
  • 86031f4034 part two: todo's in combine variants, fixes for InferredGeneticContext, and some other tests and clean-up. aaron 2010-07-05 21:07:53 +0000
  • 36edc60ccc Connected UG to the new comp track annotation system in VA. Also, when emit confidence is lower than call confidence (so that we emit records filtered with LowQual), add a corresponding FILTER header field to the VCF so that the validator doesn't complain. ebanks 2010-07-05 13:04:24 +0000
  • 3347d1ca7c part one of combining format and info header lines code into a single abstract class for Mark; plus some 'm' removals from access methods for Eric. Adding fixes for CombineVariants next. aaron 2010-07-05 05:57:58 +0000
  • e7220bc885 Variant Context simple merging routine should keep ID if one of the VCs has it ebanks 2010-07-05 01:10:15 +0000
  • 3016e1cf80 Fixes to increase robustness in vcf4 writer. We assume that only at most 1 base was clipped from beginning of allele encoding by reader, and improve the way we find if bases were clipped. We still cant deal with some corner cases, and duplicate records may follow, for example if a snp location is followed at the next base by an indel. Also, if we are reading form a 3.3 vcf and the reference is null (ie we have an insertion), the reference base is not computed correctly. delangel 2010-07-04 20:22:04 +0000
  • 07945040f8 Set VariantFiltration's JEXL engine to silent for warning messages ebanks 2010-07-04 18:11:19 +0000
  • be8740b00d Another edge case in left alignment for indels: deal with cases when insertions are ambiguously placed at ends of reads ebanks 2010-07-04 17:26:38 +0000
  • 9ec393bfce Updated md5 - vcf header line change weisburd 2010-07-02 21:02:09 +0000
  • f7593435eb Implemented decodeLoc(..) weisburd 2010-07-02 21:01:36 +0000
  • cd2e4b0a1e merging now very close to working. Bug todo in writer and vcf infrastructure. Can almost create merged snp and indel files depristo 2010-07-02 20:09:25 +0000
  • b6bdd61283 a) Fix bug when multi-base reference is homopolymeric when writing a VCF4.0 variant context: computation of number of trailing bases was incorrect and we ended up with incorrect position. b) Updated VCF4WriterTestWalker to take either VCF3 or VCF4 as inputs (this walker can also be used to convert from 3.3 to 4.0). delangel 2010-07-02 15:19:42 +0000
  • 61e2b2e39b Nearly finalize merging capabilities for CombineVariants. Support for dealing with inconsistent indel alleles at loci. Improvements to Allele and removal of addAllele to MutableGenotype. We are close to being able to merge all of 1000 genomes -- snps and indels -- into a single combined vcf depristo 2010-07-02 13:32:33 +0000
  • cab8394103 The sharding system now buffers reads, with a size determined by command-line argument. Will investigate whether/how this impacts performance on low-pass data and, if it works well, will create a more automatic version of the tool. hanna 2010-07-01 22:28:55 +0000
  • f967cae1aa tiny comment change aaron 2010-07-01 22:04:25 +0000
  • 3093a20a55 fixing VCF header format and info fields so that they propery emit the unbounded count value correctly for vcf4 or vcf3. Eric we should update the vcf4 spec page to indicate format fields are allowed to use the unbounded count as well (if this is true). aaron 2010-07-01 22:02:16 +0000
  • 61c07c6f90 Fixes for missing key values that can create null pointer exceptions when reading from 3.3-generated variant contexts. Also, chop missing genotype fields correctly from right to left delangel 2010-07-01 20:17:03 +0000
  • 87470d5fe5 Checking in a simplistic VR qscript file for posterity's sake rpoplin 2010-07-01 18:53:17 +0000
  • 255b036fb5 Variant Recalibrator MLE EM algorithm is moved over to variational Bayes EM in order to eliminate problems with singularities when clustering in higher than two dimensions. Because of this there is no longer a number of Gaussians parameter. Wiki will be updated shortly with new recommended command. rpoplin 2010-07-01 18:51:07 +0000
  • 4903d1fb4f fix for a parallelization issue: moving the creation of iterators outside of the sync block so we don't wait for RMD tracks to seek to the correct location. Thanks to Ben for providing the test case! aaron 2010-07-01 16:37:02 +0000
  • 43ca595d15 VCF headers now can be set to a particular VCF version after creation, which converts the header lines to the appropriate encoding on output. Plus some clean-up of the code. aaron 2010-07-01 05:32:14 +0000
  • 4995950d04 IndexedFastaSequenceFile is now in Picard; transitioning to that implementation. hanna 2010-07-01 04:40:31 +0000
  • b6af17b82d Rev Picard with new IndexedFastaSequenceFile patch. hanna 2010-07-01 03:05:43 +0000
  • c9d5345150 Redo StratifiedAlignmentContext to use ReadBackedPileup's stratification options. hanna 2010-07-01 02:46:05 +0000
  • dc4715c9c6 Permit empty fields in INFO and FORMAT structures - not fully tested yet but at least failing cases before now pass. Also, corrected a bug where in case we were reading 3.3 VCF's, or VCFs with no original allele encodings, we'd always print 2 bases per allele. delangel 2010-07-01 01:56:07 +0000
  • 15a6be5d50 Updated to reflect Mark's renaming binge ebanks 2010-06-30 22:48:42 +0000
  • 5f2b2d860e Final stage of renaming depristo 2010-06-30 21:39:07 +0000
  • 6e7927a47d Continuing the renaming nightmare... depristo 2010-06-30 20:25:01 +0000
  • 9d7d5f1747 Continuing the renaming nightmare... depristo 2010-06-30 20:24:27 +0000
  • aa20c52b88 deleting vcf depristo 2010-06-30 20:19:15 +0000
  • 4195fc5c4e renaming part 2... depristo 2010-06-30 20:18:11 +0000
  • 6c9da5525d renaming starting depristo 2010-06-30 20:16:51 +0000
  • b8d6a95e7a Preliminary commit of new VCFCombine, soon to be called CombineVariants (next commit) that support merging any number of VCF files via a general VC merge routine that support prioritization and merging of samples! It's now possible to merge the pilot1/2/3 call sets into a single (monster) VCF taking genotypes from pilot2, then pilot3, then pilot1 as needed. depristo 2010-06-30 20:13:03 +0000
  • a46e22ed13 Refactored ArgumentDefinition to absorb functionality from ArgumentDefinition and ArgumentTypeDescriptor. kshakir 2010-06-30 18:55:57 +0000
  • 178cf64a0c Refactored ArgumentDefinition to absorb functionality from ArgumentDefinition and ArgumentTypeDescriptor. kshakir 2010-06-30 18:37:58 +0000
  • 569456850d Mark pointed out there's differentiation in the filter field. Rolling back. chartl 2010-06-30 17:05:53 +0000
  • 52a474b27d Fixed an issue with VCF combine in sites like the following: chartl 2010-06-30 16:49:31 +0000
  • 66931d433c useful routines for R depristo 2010-06-30 16:38:49 +0000
  • 944dbb94ce Refactored and generalized the database/comp annotations in VariantAnnotator. Now one can provide comp tracks as with VariantEval (e.g. compHapMap, comp1KG_CEU) and the INFO field will be annotated with the track name (without the 'comp') if the variant record overlaps a comp site (e.g. ...;1KG_CEU;...). This means that you can now pass 1kg calls to the Unified Genotyper and automatically have records annotated with their presence in 1kg. ebanks 2010-06-30 16:37:31 +0000
  • cf910d9cc2 misc. useful updates to python library depristo 2010-06-30 16:33:32 +0000
  • 47c4a70ac1 It turns out that it is legitimately possible for there to be reads that won't overlap within a target interval for cleaning. While we don't want to attempt cleaning, we also don't want to fail. ebanks 2010-06-30 15:50:44 +0000
  • ae33d8a2f2 I just wanted one more vote. It's settled: we die. ebanks 2010-06-30 14:00:56 +0000
  • 8fb37f5f7a For Kiran: warn the user when the actual and vcf ref bases differ so that if an exception is generated later, he knows why. All: should we generate the actual exception here? Is there any reason to allow cases where the vcf record has a different ref base than the actual reference? I'd vote that we die here. Thoughts? ebanks 2010-06-30 13:56:16 +0000
  • d932322190 More necessary fixes for VCF4.0 - now results look more sensible in realistic, bigger VCF files produced by say Dindel and not just the small test VCF: - Fixed and cleaned code to produce trailing and padding bases in alleles around indels. - Deal better with missing fields. Pending: - Chopping missing fields at end of genotypes. delangel 2010-06-30 02:59:30 +0000
  • 12c0de6170 Added ability to clean using only known indels. Added integration test for it. Fixed vcf->vc conversion for indels which was busted. ebanks 2010-06-30 01:20:56 +0000
  • 610cc7ae2b Cool package trick Kiran showed me. VariantEvaluator no longer public, AAT specifies the core package even though it lives in oneoffs. Disabled so integration tests pass. chartl 2010-06-29 22:42:04 +0000
  • 4c6f4e41c6 Include making VariantEvaluator public within the package so my oneoffs can be seen (not included in previous submit specifically because I didn't want to break the build by changing anything in core...the road to hell is paved with good intentions) chartl 2010-06-29 22:26:52 +0000
  • 9ac13b8f5d Name and body change for this module to reflect local code. chartl 2010-06-29 21:45:26 +0000
  • 844cb2ed33 fixing a bug that Eric found with RODs for reads, where some records could be omitted. Sorry Eric! aaron 2010-06-29 21:38:55 +0000
  • 101c27294d Comment this guy out so we build again. (Hate it when my repository goes all funky.) chartl 2010-06-29 21:16:33 +0000
  • 3017f82550 Initial commit of items for analyzing amino acid transitions in variant eval. Blew up my subversion by coding locally while i did not have internet. I hope this doesn't bust any integrationtests since I changed no existing code but...who knows. Crossing my fingers. chartl 2010-06-29 20:57:18 +0000
  • e3fb4d5c70 Intermediate checkin, just to fix null pointer exception that happened when merging implementation with latest VCF4 decoder - field ORIGINAL_ALLELE_LIST in vc shouldn't be written in infoFields structure since this won't be output to file and there is no legal structure under this key. Base encoding for complex events is still brittle and most probably still has issues, fixes upcoming. delangel 2010-06-29 20:57:09 +0000
  • baf9479c35 An addition for Sendu since he can't seem to tell when his CountCovariate jobs die in the middle of writing the CSVs. We now write an EOF marker at the end of the covariates table and look for it when reading in the file in TableRecalibrationWalker. By default, we warn the user if the EOF marker isn't present, but we exception out if the user provides the --fail_with_no_eof_marker option. ebanks 2010-06-29 18:50:07 +0000
  • 3ca2b7374b Fixes to better deal with the "Type" and "Number" field in the INFO and FORMAT header lines in VCF4.0. We now record these fields and provide appropriate conversions. This is the first version that passes fully the VCF validator. Also, moved the flag indicating VCF4.0 to the VCFWriter constructor. delangel 2010-06-29 16:43:00 +0000
  • 801b47c6e9 For Sendu: a similar addition to the Indel Genotyper allowing it to emit a metrics file (which for now consists only of # of normal/tumor calls made) ebanks 2010-06-29 13:19:17 +0000
  • ddf87e61c2 For Sendu: optionally emit a metrics file with callability info (including number of actual calls made) from UG ebanks 2010-06-29 12:57:28 +0000
  • 929e5b9276 Fix possible null pointer exception ebanks 2010-06-29 09:01:18 +0000
  • 2953c9f069 Efficiency improvement requested by the Picard team in IndexedFastaSequenceFile: improve the memory efficiency (and loading time) of long reference sequences by better controlling the input buffer size. hanna 2010-06-29 07:22:07 +0000
  • ed71e53dd4 1) Initial complete version of VCF4 writer. There are still issues (see below) but at least this version is fully functional. It incorporates getting rid of intermediate VCFRecord so we now operate from VariantContext objects directly to VCF 4.0 output. See VCF4WriterTestWalker for usage example: it just amounts to adding vcfWriter.add(vc,ref.getBases()) in walker. delangel 2010-06-28 23:54:38 +0000
  • 4a451949ba add parallel option to target creator for masking out reads with bad mates ebanks 2010-06-28 22:13:25 +0000
  • 6a23edd911 Fix performance tests ebanks 2010-06-28 21:51:48 +0000
  • dce2c17404 Added "-bsubWait" where Queue waits for all the jobs to exit before exiting. kshakir 2010-06-28 19:52:17 +0000
  • 20f5fdbcf7 Changes to MVC to make the the header of its output VCF compliant with spec (give expected # of values for info field annotations) chartl 2010-06-28 18:33:23 +0000
  • c047232b18 Using picard for bam merging. Properties now propagate to scatter/gather functions. kshakir 2010-06-28 17:59:09 +0000
  • 62d22ff1aa adding the original allele list to a variant context (as the annotation ORIGINAL_ALLELE_LIST), in the case where the set alleles are the result of clipping. Added tests for both cases. aaron 2010-06-28 17:23:46 +0000
  • 1292c96e29 The cleaner now adds the OC (original cigar) and OS (original alignment start) tags as appropriate to reads that get realigned; this feature can be turned off. Also, improved integration tests (sorry, Kiran!). ebanks 2010-06-28 16:46:47 +0000