Skip to content

Conversation

@krdav
Copy link

@krdav krdav commented Aug 15, 2023

Clunky way of solving the request that I wrote about in this issue:
#37

Not suggesting this as the actual solution, but rather showing it so that it is clear what I am referring to in the issue. In a better solution the show_best option should probably be a boolean or yes/no and there is probably a more intelligent way of implementing it such that showhits and showalignments are not forced to be the same.

@krdav
Copy link
Author

krdav commented Aug 16, 2023

Update:
Instead of showing only top scoring sequences, I changed it such that it allows showing multiple levels e.g.
show_best=1 shows sequences with the highest score
show_best=2 shows sequences with the highest and second highest score
etc

@torognes
Copy link
Owner

Thanks for the suggested change to swipe. I haven't worked on the swipe code for a long time, but will try to find time to look at this soon.

@krdav
Copy link
Author

krdav commented Aug 21, 2023

Thank you so much!
I understand if you got other priorities, not working on the swipe code.
But Swipe just happen to be very fast - in my experience a lot faster than other SW tools/libraries.

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.

2 participants