-
Notifications
You must be signed in to change notification settings - Fork 48
Description
Thought about this due to this comment from #322
This commit adds new methods for findnext and findprev for SeqOrView with known alphabets,...
It's only implemented for known alphabets because new alphabets may overload == in surprising ways, which makes the bitparallel ops invalid.
Possibly related to #296 and #140 ?
I know we've had this conversation before, but couldn't find it in issues - on the one hand, it would be cool if it were possible for people to come up with their own alphabets and be able to take advantage of all of these optimizations. On the other hand, I can't really think of what those alternative alphabets might be, and if someone is advanced enough to want them, maybe they can figure it out.
On the third hand 🐙, it might make it more feasible for others to provide package maintenance if all of this stuff is better documented, even if no extensions are desired. I am mindful of your development time though @jakobnissen, and I know documentation is often a low priority (you do a great job prioritizing it for user-facing stuff).