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.

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

Include query sequences in read_bam_full #4

Merged
merged 1 commit into from
Jun 9, 2021

Conversation

timodonnell
Copy link
Contributor

Fixes #3

@endrebak endrebak merged commit 44234f9 into pyranges:master Jun 9, 2021
@endrebak
Copy link
Collaborator

endrebak commented Jun 9, 2021

I had actually gotten the sequences in my cython code, but forgotten to include them in my python code.

@endrebak
Copy link
Collaborator

endrebak commented Jun 9, 2021

0.0.7 out on pypi. Thanks!

Are you using bamread as a standalone library? I guess it isn't very user-friendly, but I thought people would be using it from pyranges so I haven't polished it much.

@timodonnell
Copy link
Contributor Author

Thanks! Yeah, I've been using it standalone. I think there is space for a reasonably performant way to read BAMs into a pandas dataframe without a big (and ever changing) API like pysam. This project could be that!

@endrebak
Copy link
Collaborator

endrebak commented Jun 9, 2021 via email

@timodonnell
Copy link
Contributor Author

Sure, I'll make a ticket with thoughts on an API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature request: include sequences
2 participants