Skip to content

Adding GATK concordance script (connected to #167)#173

Open
heuermh wants to merge 8 commits into
BD2KGenomics:masterfrom
heuermh:concordance
Open

Adding GATK concordance script (connected to #167)#173
heuermh wants to merge 8 commits into
BD2KGenomics:masterfrom
heuermh:concordance

Conversation

@heuermh

@heuermh heuermh commented Mar 14, 2016

Copy link
Copy Markdown
Member

Work towards #167, still needs to be tested.

@heuermh heuermh changed the title first checked in Adding GATK concordance script Mar 14, 2016
@hannes-ucsc hannes-ucsc changed the title Adding GATK concordance script Adding GATK concordance script (resolves #167) Mar 14, 2016
@hannes-ucsc

Copy link
Copy Markdown
Contributor

Fixed title and added needs work label. Let me know if any of these changes are inaccurate.

return parser


# copied from germline.py

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NONONONONONONONONONONONONONONO! No copying!

:)

@fnothaft fnothaft changed the title Adding GATK concordance script (resolves #167) Adding GATK concordance script Mar 16, 2016
@fnothaft

Copy link
Copy Markdown
Contributor

I've removed the "Resolves #167" title, as we still need liftover code. @heuermh will you add VCF liftover to this, or open a new PR with it? I'd probably prefer a new PR.

@heuermh

heuermh commented Mar 16, 2016

Copy link
Copy Markdown
Member Author

Sure, in a new PR

@hannes-ucsc hannes-ucsc changed the title Adding GATK concordance script Adding GATK concordance script (connected to #167) Mar 16, 2016
@hannes-ucsc

Copy link
Copy Markdown
Contributor

In that case the connected to linkage is recommended. It's used by Waffle to stack PRs and issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants