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

add .interval_list file parser #686

Closed
wants to merge 6 commits into from
Closed

add .interval_list file parser #686

wants to merge 6 commits into from

Conversation

ryan-williams
Copy link
Member

fixes #685

@fnothaft
Copy link
Member

Jenkins, retest this please.

@fnothaft
Copy link
Member

Thanks @ryan-williams! I've manually merged this, after rebasing and squashing down to two commits:

  • One for the interval list work: 35fbbb6
  • One for the deprecation warning: d5255ae

@fnothaft fnothaft closed this May 31, 2015
@ryan-williams
Copy link
Member Author

amusing P.S. to this PR: while searching for my own code from this PR today I discovered IntervalListReader lying around, which wraps an HTSJDK IntervalList parser.

AFAICT it doesn't do anything with any of the dbxref/attr metadata that my .interval_list files have, so I think I prefer my impl, and all is not lost (or double-gained).

@ryan-williams ryan-williams mentioned this pull request Jun 2, 2015
Closed
@fnothaft fnothaft mentioned this pull request Sep 7, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support parsing an RDD[Feature] from a Picard .interval_list file
2 participants
@ryan-williams @fnothaft