Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Port interval-parsing code from GATK, and hook up -L and -XL arguments to the engine #98

Closed
droazen opened this issue Jan 13, 2015 · 6 comments · Fixed by #164
Closed

Port interval-parsing code from GATK, and hook up -L and -XL arguments to the engine #98

droazen opened this issue Jan 13, 2015 · 6 comments · Fixed by #164
Assignees
@lbergelson
Labels
Engine
Milestone
Aye-aye

Comments

@droazen
Copy link
Contributor

droazen commented Jan 13, 2015

No description provided.

@droazen droazen added the Engine label Jan 13, 2015
@droazen
Copy link
Contributor Author

droazen commented Jan 13, 2015

This may require us to decide between GATK-style interval files and Picard-style interval files.

@jmthibault79
Copy link
Contributor

Or allow both, converting to an internal representation?

@lbergelson
Copy link
Member

We should definitely support both styles.

@droazen
Copy link
Contributor Author

droazen commented Jan 13, 2015

That is an option, but there have been requests to standardize on one format

@akiezun akiezun mentioned this issue Jan 13, 2015
Closed
@akiezun akiezun modified the milestone: Aye-aye Jan 13, 2015
@akiezun
Copy link
Contributor

akiezun commented Jan 13, 2015

@yfarjoun @nh13 we'd like your opinion on this issue.

@droazen
Copy link
Contributor Author

droazen commented Jan 13, 2015

Closed as dupe of #4

@droazen droazen closed this as completed Jan 13, 2015
@lbergelson lbergelson mentioned this issue Feb 6, 2015
Merged
lucidtronix added a commit that referenced this issue Jul 27, 2021
@lucidtronix
Daisy chain TensorMap parents to learn trees (#98)
254369c 
parents are tmaps not string
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lbergelson lbergelson

Labels
Engine
Projects
None yet
Milestone
Aye-aye
Development

Successfully merging a pull request may close this issue.

Enabling -L system broadinstitute/gatk
4 participants
@droazen @akiezun @jmthibault79 @lbergelson