[Feature request] Estimated Total Knowledge over time

Any chance that you will implement it in Anki natively?

Possibly. As I’ve said I should have done it natively from the start.

SSE should be updated now :+1::partying_face::sleeping:

I guess it’s caused by the same problem mentioned here: Set Due Date doesn't update the interval of card

So the Estimated total knowledge in Card Retrievability section is not perfectly accurate.

In addition, suspended cards are excluded in Card Retrievability section. It also induced difference.

Btw, if I understand it correctly, the last value of “Estimated total knowledge over time” shows the sum of retrievability of the yesterday, right?

Nice catch! It’s always the obvious stuff I miss :person_facepalming:.

Ok I’ll stop aiming for the estimated total knowledge. I feel like there is something else wrong with the addon because the stabilities of the cards the addon generates don’t line up with the stored stabilities in Anki.

const expected = last_stability[num_cid].toFixed(2)
const actual = JSON.parse(cards_by_id[num_cid].data).s.toFixed(2)
if (expected != actual) {
    inaccurate_cids.push({ cid: num_cid, expected, actual })
}

Is that a problem?

Could you test it with one card in a new deck? I guess it would help us locate the problem.






image

Is this ok? You can see the difference in the forgetting curve and the displayed graph.

As the graph is an estimate anyway it may not be a problem worth solving?

I don’t know why it didn’t reach 100% here.

That’s because of the way the candlestick works. It displays the beginning and end values for that period of time.

Fine. I will dive into the code and try to solve it.

Now I’m even more convinced that candlestick charts are not suitable for Anki

Ok I’ll try to make it a line graph. Never really liked the candlestick anyway it was just convenient.


Same card as earlier:

Much better. I’ll probably give an option to replace the load chart like this as well.

I just noticed that changing the parameters in the deck config doesnt affect the graph…

Of course, it should affect the graph. What’s the problem?

I was using config.fsrsWeights instead of config.fsrsParams5.

Fixing that still didnt entirely fix the stabilites though I don’t think :frowning:

I guess I find the same problem you mentioned?

Edit: I fixed that and still found that the consistency happened after the 2nd review:

I guess it’s problem of ts-fsrs.

Edit: I located the problem and reported it to the maintainer of ts-fsrs. It will be fixed soon.

pr:Fix/wrong calc elapsed days by ishiko732 · Pull Request #144 · open-spaced-repetition/ts-fsrs · GitHub

@A_Blokee would you mind testing it?

Before:
image
Now:

That fixes it for a lot of cards but some are still wrong
It may have something to do with rollover?

I guess so. Let’s discuss it in the PR.