search

xquercus / load-balanced-scheduler

GNU General Public License v3.0
10 stars 9 forks source link

Choose a random interval from all lowest intervals found #1

Closed DrewSSP closed 5 years ago

DrewSSP commented 6 years ago

I noticed that the addon always chooses the latest possible option for best_ivl, which will almost always push scheduling to be later than Anki naturally wants it to be. This, in turn, will increase the rate of wrong answers and ultimately just make reviewing harder than it needs to be.

This proposed change collects the lowest num_cards values for every interval between min_ivl and max_ivl, then stores those values inside a list called acceptable_ivls. It chooses a random item from within that list for best_ivl.

Logging functions were modified rather liberally to accommodate the changes.

xquercus commented 6 years ago

This looks great. I used it on my own deck today. Within the next couple of weeks I'm hoping to put together a 2.0 compatible package and will probably include it then -- for both 2.0 and 2.1. Thanks!

matrus commented 5 years ago

Any chance this being merged in an upcoming version?

xquercus commented 5 years ago

Thinking things over this probably won't be merged. It adds slightly to the code complexity and I'm not sure I want to do that.