Conversation
9c36761 to
d58c5f2
Compare
|
This is more or less ready per se. But unfortunately prime counting is so intimately (and impenetrably) connected to the existing Eratosthenes sieve, that I cannot decomission the latter. That said, this PR is on hold until I disentangle this knot. |
|
Hey @Bodigrim,
I understand that merging this as is without removing the earlier sieve code would just clutter the codebase (even further), but if it's already working with good performance, wouldn't merging it now and making a new PR that proceeds with decoupling the |
|
My main goal here is to make source code more maintainable. I can even agree to worsen performance mildly. It might happen that I'd never untangle |
|
@Bodigrim can't say I don't understand. It's already difficult enough maintaining the package as it is - I remember trying to migrate from |
95c2345 to
09a9d5f
Compare
Closes #157
It is already competitive with the existing sieve and is 10 times simpler, but I need to merge more performance patches into https://github.com/Bodigrim/bitvec to make this code really shine.