xquercus / load-balanced-scheduler

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

Fix load balancer not working for cards with short intervals in the v2 scheduler #13

Closed kuroahna closed 2 years ago

kuroahna commented 2 years ago

This should fix the issue in https://github.com/xquercus/load-balanced-scheduler/issues/11

I also made a fork of AnkiDroid, porting this addon to Android, and added some JUnit tests which you can find here: https://github.com/kuroahna/Anki-Android/commit/1268be0709b260b783842b804831a1f7b8f1871d

I think it'd be good to add some python unit tests here as well so that we don't face regression. Relying on manual debugging with the logging is bad IMO, but I'm not too sure on how to add them. AnkiDroid surprisingly has better documentation, and well, Java being easier to work with than Python, so it was fairly easy to write some tests there.

I have manually tested these changes in v2.1.48 and things are working as expected as far as I can tell.