Closed MillironX closed 2 years ago
Base: 88.50% // Head: 87.60% // Decreases project coverage by -0.90%
:warning:
Coverage data is based on head (
fb312a5
) compared to base (6d96e79
). Patch coverage: 83.65% of modified lines in pull request are covered.:exclamation: Current head fb312a5 differs from pull request most recent head 797c566. Consider uploading reports for the commit 797c566 to get more accurate results
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Types of changes
This PR implements the following changes:
:clipboard: Additional detail
The idea for these getter functions first came up in the discussion at https://github.com/BioJulia/BioAlignments.jl/discussions/80#discussioncomment-2959292. These functions are intended to be user-friendly band-aids over the disjointed API between
Alignment
,AlignedSequence
,PairwiseAlignment
, andPairwiseAlignmentResult
. There are five new methods, with one new exported function.alignment(::AlignedSequence) -> Alignment
alignment(::PairwiseAlignment) -> Alignment
sequence(::AlignedSequence) -> LongSequence
sequence(::PairwiseAlignment) -> LongSequence
sequence(::PairwiseAlignmentResult) -> LongSequence
Note that
alignment(::PairwiseAlignmentResult)
already returns aPairwiseAlignment
. I opted for liberal documentation over breaking the API.I added extra documentation and tests in the form of doctests. I didn't add any additional tests to the regular test suite, as getter function tests are pretty redundant.
My hope is that this closes out the feature list of BioAlignments v2, and we can get #44 merged and move on to BioAlignments v3 after this.
:ballot_box_with_check: Checklist
docs/src/
.[UNRELEASED]
section of the manually curatedCHANGELOG.md
file for this repository.