Closed eshapard closed 6 years ago
Updated code... I think this will fix the issue, but have not tested it.
Seems working now. One more thing - it always make progress on 'step' value of good answers. So if i have step = 2 and reach 80% then after 1st good answer nothing happen. And after 2nd it sets 2 ticks at once. Intuitively progress should be 2x faster, so same amount of pics with 2x less good answers, not 2x more pics with same amount of answers. But it's not really bothering tbh. I don't know if you've ever considered 'solid' progress bar. I think it should work better - visual feedback is immediate no matter what setting you have. And such problems wouldn't exist. And sorry again for mention multiple things in one thread. Fix for bug looks like working :) Thanks for help.
I did consider a more granular approach, but it would be much harder to implement, and the benefit just doesn't outweigh the increased complexity in my opinion.
On Mon, Sep 18, 2017 at 11:25 AM, Łukasz Rosiek notifications@github.com wrote:
Seems working now. One more thing - it always make progress on 'step' value of good answers. So if i have step = 2 and reach 80% then after 1st good answer nothing happen. And after 2nd it sets 2 ticks at once. Intuitively progress should be 2x faster, so with 2x less good answers. But it's not really bothering tbh. I don't know if you've ever considered 'solid' progress bar. I think it should work better - visual feedback is immediate no matter what setting you have. And such problems wouldn't exist. And sorry again for mention multiple things in one thread. Fix for bug looks like working :) Thanks for help.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/eshapard/AnkiHabitica/issues/55#issuecomment-330313524, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5-KtdjNVgtVsW_KJOdxVyHOwELN2Ojks5sjrWzgaJpZM4PbOQM .
This seems to be because the code was not tested much with step > 1
Will adjust make_habit_progbar function to try to account for this.