Also, can you make the x axis in dates? Like, November 2024.
As a sidenote / question: Is there a reason why the graphes in anki have no x and y axis titles? Normally in maths and sciences you’re required to add those so that it’s obvious what the graph is showing.
E.g.
Here the x axis would be something like date or x days ago and the y axis something like amount of cards. The old graphs apparently had that, see the picture called old here.
1 Like
I assume it’s just a stylistic choice, since all of that units information is captured (and with specificity) in the hover/tap pop-up. It might seem unusual from a math/science perspective – but from a data-presentation perspective, I don’t mind that it gives the graph a chance to stand alone as a visual.
1 Like
I guess that makes sense.
I’d still prefer the “scientific way” since it allows me to see all the important information at a glance without having to hover over every graph. And while I personally can deduce the meaning of the axis most of the time, I don’t think it’s that obvious for everyone.
But that’s a discussion for another topic. I’ll open one in the near future if there isn’t already one.
2 Likes
I guess part of it might be the requirement that they work well on small screens.
On a mobile phone the axis labels may eat a significant amount of the available space.
If that is the issue you could make them responsive so the labels only show when you have plenty of screen space.
1 Like
Yup, we already have text overflow issues in this screen and in the simulator too where we have a label for y-axis.
It actually looks better without the labels according to me.
I got the memorised data from the simulator showing. Looks like it doesn’t include the memorised value from the cards before its first simulated review though making the graph somewhat useless.
branch:
2 Likes
Oops. I find a bug when I analyze this problem… We forgot to update memorized_cnt_per_day when learning new cards in the last refactor.
Edit: Yep, this problem does exist because we only update the memorized_cnt_per_day when the card is reviewed in the simulator. I will submit a PR to improve it.
PR:
2 Likes
Can you rename “count” to “reviews”? The y-axis title in the image is also “memorised count” which makes it a bit confusing because it’s not clear what “count” is.
And if you’re willing to make some other UI changes, the radio buttons and the “Simulate”/“Clear last simulation” should have some gap between them.
2 Likes
I invited you to join OSR, and I hope you can help me review the code.
3 Likes
Well, now I want a native “Estimated total knowledge over time” (historical, not simulated) even more 
