Commit Graph

13320 Commits (267603f9a95843fb00f751102a3b973c5dd662d9)

Author SHA1 Message Date
Eric Banks 267603f9a9 Merge pull request #589 from broadinstitute/ldg_SelVarXsampleFile
Added check to make sure file passed in with sample IDs is valid (used i...
2014-04-04 15:56:16 -04:00
Laura Gauthier ff25b656e1 Added check to make sure file passed in with sample IDs is valid (used in SelectVariants) -- throws UserException. Corresponding test checks for UserException. 2014-04-04 15:38:50 -04:00
Laura Gauthier 06d78ba068 Expanded documentation to include description of which callsets are being compared in what order and more definitions 2014-04-04 10:35:53 -04:00
Eric Banks 9be07e0838 Merge pull request #588 from broadinstitute/eb_fix_ir_exception
IndelRealigner throws a user error when it encounters reads with I opera...
2014-04-04 10:11:51 -04:00
Eric Banks 7174f8cfeb IndelRealigner throws a user error when it encounters reads with I operators greater than the number of read bases.
Added test to ensure it works.
2014-04-03 18:16:24 -04:00
Geraldine Van der Auwera 890f4e8873 Merge pull request #586 from broadinstitute/eb_allow_users_to_specify_iupac_sample
Slightly modifying the way to use the IUPAC ambiguity codes in the Fasta...
2014-04-03 09:29:56 -04:00
Eric Banks 0b73573abc Slightly modifying the way to use the IUPAC ambiguity codes in the FastaAlternateReferenceMaker.
Previously it required you to create a single sample VCF and then to pass that in to the tool, but
Geraldine convinced me that this was a pain for users (because they usually have multi-sample VCFs).
Instead now you can pass in a multi-sample VCF and specify which sample's genotypes should be used
for the IUPAC encoding.  Therefore the argument changed from '--useIUPAC' to '--use_IUPAC_sample NA12878'.
2014-04-02 21:34:25 -04:00
Eric Banks 6bba8d7147 Merge pull request #585 from broadinstitute/ks_variantqc_patch
Resuscitated from git and copy/pasted in old gsalib methods need for the private script variantCallQC.R to run, for now.
2014-04-02 16:48:42 -04:00
Khalid Shakir 0647824e75 Resuscitated from git and copy/pasted in old gsalib methods need for the private script variantCallQC.R to run, for now. 2014-04-03 04:22:11 +08:00
Valentin Ruano Rubio 45c192bb6d Merge pull request #580 from broadinstitute/vrr_graphbase_infinite_likelihoods_reprise
Fixed bug using GraphBased due to infinite likelihoods resulting from th...
2014-04-02 00:45:17 -04:00
Valentin Ruano-Rubio 84711b8e90 Fixed bug using GraphBased due to infinite likelihoods resulting from the calculation of alignment cost of very long insertion or deletions (done in linear scale)
Stories:

  https://www.pivotaltracker.com/story/show/66263868

Bug:

  The problem was due to the way we were calculating the fix penalty of a large deletion or insertion. In this case we calculate the alignment likelihood of the portion
  or read or haplotype deletion as the penalty of that deletion/insertion without going through the full pair-hmm process. For large events this resulted in a 0 in
  in linear scale computations that ins transformed into an infinity in log scale.

Changes:

  - Change to use log10 scale for calculate those penalties.
  - Minor addition of .gitignore to hide ./public/external-example/target which is generated by the building process.
2014-04-01 16:14:52 -04:00
droazen c0286853b7 Merge pull request #584 from broadinstitute/dr_update_queue_test_script_for_naming_change
Update queue test runner script for upcoming naming changes
2014-04-01 11:52:22 -04:00
David Roazen ef8f91a5be Update queue test runner script for upcoming naming changes
Use both the old and new names for now, until the transition
is complete.
2014-04-01 11:49:55 -04:00
jmthibault79 8703bd7ad4 Merge pull request #583 from broadinstitute/jt_tabix
Create Tabix indices for block-compressed VCFs
2014-03-31 16:17:25 -04:00
Joel Thibault 70fe7f72f1 Return a TabixIndexCreator for appropriate file types
[Fixes #68291082]
2014-03-31 16:15:34 -04:00
Joel Thibault ab5634cbac Test that a Tabix index is created for block-compressed output formats
- Replace .idx and .tbi with appropriate constants
2014-03-31 14:36:48 -04:00
Joel Thibault a2d40c84ba Keep the list of zipped suffixes in sync with Variant 2014-03-31 14:36:41 -04:00
Joel Thibault a2cd9703fa Rev Picard 1.110.1773 2014-03-31 14:15:06 -04:00
Eric Banks 821fbe7260 Merge pull request #582 from broadinstitute/vrr_hc_bugfixes_dangling_heads
Fix loss of key alternative haplotypes due to a change on threading star...
2014-03-31 10:42:08 -04:00
jmthibault79 313d20d849 Merge pull request #581 from broadinstitute/jt_picard
Rev Picard because it's needed by another branch
2014-03-30 11:05:50 -04:00
Joel Thibault 2049eb1658 Rev Picard 1.110.1763
- SamPairUtils migrated in Picard r1737
- Revert IndelRealigner changes made in commit 4f4b85
-- Those changes were based on Picard revision 1722 to net/sf/picard/sam/SamPairUtil.java
-- Picard revision 1723 reverts these changes, so we also revert to match
2014-03-30 09:33:57 -04:00
Valentin Ruano-Rubio 258b2bce28 Fix loss of key alternative haplotypes due to a change on threading start policy required when recovering dangling heads.
Story:

  - https://www.pivotaltracker.com/story/show/67601310

Change:

  - Unless recover-danging-heads is active, the threading starting location policy is the original one. i.e. just at already existing unique kmer vertices.

Tests:

  - HaplotypeCallerIntegrationTest#testMissingKeyAlternativeHaplotypesBugFix
2014-03-29 22:40:26 -04:00
MauricioCarneiro 5abb7ea2db Merge pull request #579 from broadinstitute/rp_fix_DP_annotation
Fix for dropping of reference sample depth in the DP annotation.
2014-03-24 15:49:20 -04:00
Ryan Poplin 6566dd6ca9 Fix for dropping of reference sample depth in the DP annotation.
-- In the case of hierarchical merge we can't assume that we have only one genotype.
-- Removed use of deprecated VC annotation access functions.
2014-03-24 14:01:50 -04:00
Ryan Poplin c61a791914 Merge pull request #578 from broadinstitute/eb_trivial_fix_to_IR
Fix for reads that are all insertions (e.g. 50I) and causing the IndelRe...
2014-03-21 15:24:58 -04:00
Ryan Poplin b8581d7d3a Merge pull request #576 from broadinstitute/rp_fix_AssessNA12878_dropping_contigs
Bug fix in AssessNA12878 when working with more than one contig.
2014-03-21 15:23:59 -04:00
Eric Banks 32a96e3ab3 Fix for reads that are all insertions (e.g. 50I) and causing the IndelRealigner to error out. 2014-03-21 15:01:34 -04:00
Ryan Poplin dd1b0a48db Bug fix in AssessNA12878 when working with more than one contig.
-- SmartSiteIterator needs to know to span across Chunks when iterating by polling the Chunk list
-- Added KB test to test for this case
-- Removed the maxSites argument in ExtractConsensusSites because it is counterintuitive and not useful.
2014-03-21 14:52:10 -04:00
Ryan Poplin 69eaf7c82d Merge pull request #577 from broadinstitute/eb_minor_fixes_for_fragment_utils
Fixed docs for method and fixed the edge case optimization to properly u...
2014-03-21 14:01:44 -04:00
Ryan Poplin ce39fcd8a3 Merge pull request #575 from broadinstitute/eb_various_fixes_for_gvcfs
Eb various fixes for gvcfs
2014-03-21 09:47:08 -04:00
Eric Banks 0d82a70633 Fixed docs for method and fixed the edge case optimization to properly use equals() on Integers.
Shouldn't affect actual results at all.
2014-03-20 15:55:09 -04:00
Eric Banks 7c8ce3cd6a Several improvements to GenotypeGVCFs: --includeNonVariantSites now actually works and we propagate AD to hom ref samples 2014-03-20 00:35:54 -04:00
Eric Banks 824983af1d Enable CombineGVCFs to process gVCFs that were created with basepair resolution. 2014-03-19 19:23:05 -04:00
Eric Banks 3b1c337401 Have CombineVariants throw a UserError when trying to combine GVCFs from the HaplotypeCaller.
Was previously throwing an IllegalArgumentException (in the wrong place in the code).
Error message tells users to use CombineGVCFs.
2014-03-19 19:11:40 -04:00
Ryan Poplin 7117bebb5e Merge pull request #572 from broadinstitute/rp_fix_manual_reviews_fix
Forgot to change the padding ref base as well.
2014-03-19 13:58:58 -04:00
Ryan Poplin a3aa68e626 Forgot to change the padding ref base as well. 2014-03-19 13:58:21 -04:00
Ryan Poplin c737b8ed1e Merge pull request #571 from broadinstitute/rp_fix_manual_reviews
Fixing the reference base in one of the manual review files.
2014-03-19 13:49:51 -04:00
Ryan Poplin 523fd40a07 Fixing the reference base in one of the manual review files. 2014-03-19 11:08:06 -04:00
Ryan Poplin 3f326b14be Merge pull request #570 from broadinstitute/rp_assessNA12878_arg_docs_fix
Small argument docs fix in AssessNA12878.
2014-03-19 09:01:23 -04:00
droazen 7b38019199 Merge pull request #569 from broadinstitute/vrr_speedup_integration_test
Reduce runtime of very long integration test
2014-03-18 23:39:16 -04:00
Valentin Ruano-Rubio 905b6066b2 Reduce runtime of very long integration test 2014-03-18 21:48:13 -04:00
Eric Banks 14eb0a8a30 Merge pull request #566 from jsilter/master
Improvements to na12878kb
2014-03-18 15:16:20 -04:00
droazen cec4ff3a2a Merge pull request #568 from broadinstitute/dr_fix_UtilsUnitTest
Fix typo in UtilsUnitTest data provider name
2014-03-18 14:31:14 -04:00
David Roazen e549f4a9d2 Fix typo in UtilsUnitTest data provider name
This is currently my leading suspect for the cause of the
intermittent NoSuchElementException errors on master, since
the maven surefire plugin seems unable to handle errors in
TestNG DataProviders without blowing up.
2014-03-18 11:52:29 -04:00
Ryan Poplin a02383fc6a Small argument docs fix in AssessNA12878. 2014-03-18 10:07:14 -04:00
David Roazen 4ba72d43cf Re-enable GATKRunReportUnitTest
This test is not, as I had initially thought, the cause of the
maven errors. Our master branch is failing intermittently
regardless of whether this test is enabled or disabled.

This reverts commit 45fc9ff515eec8d676b64a04fb34fb357492ff84.
2014-03-18 09:53:41 -04:00
David Roazen 3cd8158bed Merged bug fix from Stable into Unstable 2014-03-18 03:01:28 -04:00
David Roazen cfc45fdc0b Disable GATKRunReportUnitTest
These tests pass individually and as part of complete test suite runs,
but cause an intermittent NoSuchElementException in maven when the
unit tests are run on their own. Disabling these tests until the
cause of this can be identified.
2014-03-18 02:57:19 -04:00
David Roazen afa6abe554 Temporarily disable GATKRunReportUnitTest in unstable while maven issues are worked out
This test passes when run individually, as part of the commit tests, or as
part of the package tests. However, when running the unit tests in isolation
it causes maven/surefire to throw a NoSuchElementException.

This is clearly a maven/surefire bug or configuration issue. I will re-enable
this test on a branch as Khalid and I try to work through it.
2014-03-18 01:28:28 -04:00
David Roazen 2d8653f493 Update pom versions to mark the start of GATK 3.2 development 2014-03-18 01:18:59 -04:00