Add basic functionality to read indexed FASTA files#165
Closed
landesfeind wants to merge 11 commits into
Closed
Conversation
dlaehnemann
requested changes
Jun 8, 2020
Member
dlaehnemann
left a comment
dlaehnemann
left a comment
There was a problem hiding this comment.
@brainstorm requested a review, so here it comes. Let me know, if I should have another look.
Contributor
Author
|
Dear @dlaehnemann, thank you very much for the review! |
brainstorm
added a commit
that referenced
this pull request
Aug 4, 2020
* Cherry pick the pull request for faidx #165 (review) * Cargo formatting * Update faidx documentation for clarification * Implement faidx_open test * Run actions on faidx branch * Remove faidx from github actions * Activate CI for faidx branch * update osx bindings with faidx support * update linux bindings for faidx * Drop faidx branch from CI actions As requested by @brainstorm (see #214 (comment)) * Migrate faidx to use thiserror instead of snafu (#214 (comment)) * Use i64 in faidx and return byte array Internal C method faidx_fetch_seq64 uses i64 as type to index into large FASTA sequences. The fetch function now returns a byte array instead of a String to be more efficient. A new method was added to automatically convert to a String for convenience. * Formatting using cargo fmt * Use usize-type for position paramter in faidx The return type of the faidx functions is changed to a Result capturing a potential failure in converting the usize into a i64 (hts_pos_t in the C API). * Correct typo in faidx-error: to -> too Co-authored-by: Patrick Marks <pmarks@gmail.com> Co-authored-by: Patrick Marks <patrick@10xgenomics.com> Co-authored-by: Roman Valls Guimera <brainstorm@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
this pull request adds basic FAIDX functionality. Importantly, not all functions are supported/wrapped but enough for my use case.
I tried to follow your coding conventions but please don't hesitate to point out anything which can be improved.
Best,
Manuel