Skip to content
New issue

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

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

Already on GitHub? # to your account

sort/maintain-list-order: maintain original order when when scores are tied #85

Merged
merged 2 commits into from
May 8, 2021

Conversation

AndreZiviani
Copy link
Contributor

@AndreZiviani AndreZiviani commented May 3, 2021

When multiple slices have the same SmithWaterman score the order in which they appear are not preserved, this PR improves the sorting function to keep the original order

Copy link
Owner

@ktr0731 ktr0731 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution!
I missed the stable order between the same two values.

I submitted one suggestion.

@ktr0731 ktr0731 assigned ktr0731 and unassigned ktr0731 May 7, 2021
@ktr0731 ktr0731 added the enhancement New feature or request label May 7, 2021
@codecov
Copy link

codecov bot commented May 7, 2021

Codecov Report

Merging #85 (124ac36) into master (3920822) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
+ Coverage   87.10%   87.13%   +0.02%     
==========================================
  Files           5        5              
  Lines         543      544       +1     
==========================================
+ Hits          473      474       +1     
  Misses         57       57              
  Partials       13       13              

@AndreZiviani
Copy link
Contributor Author

That else was an else if at some point and I forgot to remove it :P

Copy link
Owner

@ktr0731 ktr0731 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Thanks for your contribution!

@ktr0731 ktr0731 merged commit 0a80152 into ktr0731:master May 8, 2021
@AndreZiviani AndreZiviani deleted the sort/maintain-list-order branch May 17, 2021 13:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants