iamjustkoi / StudyTimeStats

Customizable statistics add-on for Anki.
https://ankiweb.net/shared/info/1247171202
MIT License
18 stars 4 forks source link

ETA Field Shows 0 min on Overview Page of a Custom Study Session #50

Open CHA0S-KING opened 7 months ago

CHA0S-KING commented 7 months ago

Bug/Issue Description For some reason, my ETA always shows as 0:00 hrs in my custom studies. Am I doing something wrong or is this a bug?

Screenshot 2024-02-10 122839

To Reproduce Steps to reproduce:

  1. Create a custom study using cards from multiple subdecks/tags
  2. Manually edit custom study options to include both new and due cards (e.g., "is:new or is:due")

For reference, this is my custom study in Anking:

is:new "deck:1 - AnKing" (tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Cardiovascular OR tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Neurology OR tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Psychiatry_/_Psychology)

OR

is:due "deck:1 - AnKing" (tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Cardiovascular OR tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Neurology OR tag:#AK_Original_Decks::Step_1::Zanki_Step_Decks::Zanki_Psychiatry_/_Psychology)

  1. Enable the ETA feature in the add on

Screenshot 2024-02-10 122922

iamjustkoi commented 7 months ago

Hey! Sorry this's poppin up for you.

It seems like it might have to do with it being a custom study. To be honest, I don't think I set up the add-on to work too well with times for any deck-specific macros (ETA's one of them, I'm pretty sure, unfortunately).

I'll defs try and see what I can do for the next update, though, and thanks for the report!

CHA0S-KING commented 7 months ago

Ah okay, thank you for the prompt reply, I'd appreciate that fix!

A couple other things:

1) I noticed that when you click "rebuild" for the custom study, it completely resets all the stats to 0, including all the "total" macros. Can this be fixed so that the "total" macros remain untouched (assuming the custom study filter hasn't changed), and act as a "running total"?

2) Very minor and appearance related but can you change the time displays from a clock appearance (43:40) to a traditional appearance (43 hrs 40 min). Perhaps you can also add more units for the user to include (days, seconds, etc) in time based macros. Right now it seems to only be hours and minutes. The macros can then be consolidated into just one macro that is "Total Time" rather than "Total hours". Just a thought!

3) Is there a guide posted anywhere that explains the functionality of each macro and how to use them in depth? If not, would you be able to make one?

I know it's a lot, but those are just some thoughts! Really loving your work so far, keep it up!

Cheers!


From: JustKoi @.> Sent: Sunday, February 11, 2024 9:59:25 AM To: iamjustkoi/StudyTimeStats @.> Cc: CHA0S-KING @.>; Author @.> Subject: Re: [iamjustkoi/StudyTimeStats] ETA Field Shows 0 min on Overview Page of a Custom Study Session (Issue #50)

Hey! Sorry this's poppin up for you.

It seems like it might have to do with it being a custom study. To be honest, I don't think I set up the add-on to work too well with times for any deck-specific macros (ETA's one of them, I'm pretty sure, unfortunately).

I'll defs try and see what I can do for the next update, though, and thanks for the report!

— Reply to this email directly, view it on GitHubhttps://github.com/iamjustkoi/StudyTimeStats/issues/50#issuecomment-1937778018, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BGB7TFLMQP5YQKAUNTMAPNTYTDME3AVCNFSM6AAAAABDC3TOKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZXG43TQMBRHA. You are receiving this because you authored the thread.Message ID: @.***>

iamjustkoi commented 6 months ago

Hey! Thanks so much for the kind words and awesome write-up! ^^

All good stuff to know and megs appreciate it, as always.