[GUI Plugin] Reading Goal

[thiago.eec]

Originally Posted by ownedbycats

And I guess even if I did insert the starting date, it'd still have a massive jump. Hmm.

There's three things I can think of:

1. Average out the pages between the records instead of putting all the pages in on the date of new record.

2. A way to exclude pages from certain books?

3. Maybe ignore books that only have one date?

I'm working on a new dialog for editing the book's records. It's not saving the edits yet, but you can see how it's gonna work with the test version attached. Select the book and click on the new button on the right side of the 'Edit reading goal' dialog. I think this solves your problem and gives us a general way to deal with wrong records.

Originally Posted by Feoras

I would like to come back to this request, though except "alongside" I would like to have the option to change the main statistic view to show pages instead of the book count.
Sadly I do not read that many books a month, so having the pages read per month makes the statistic more useful to me.

Idea:
* Similar to `read_books_month_count` add a statistic for pages read per month.
* Have a config checkbox to change this setting.
* `get_month_stats` returns either the current count or the new pages count based on the checkbox

Working prototype git diff attached (inside zip).
(Might need some logic to now show the read books on month hover instead of pages (in `get_month_html`)


---
Another idea would be to expand the statistics view to have one with book read count and another next to it (like to the right) with the page count. This would probably the "alongside" request as original stated.


Anyway, thanks for considering any of this!

Thanks for the help!
In my testings, the code you provided got the wrong page count, though. For one thing, it was working with accumulated values:

read_pages_month_count[month] += read_page_count - last_year_pages_count

Note that this line is inside a 'for' loop, iterating over all books in the goal, so the 'read_page_count' refers to the whole year.
To get only the month values, I used the existing 'daily_page_count' method.

Anyway, test the version attached.

P.S.: Note what I wrote above, replying to @ownedbycats. There is an unfinished feature in this test version, but it doesn't affect any other feature or modify the database.

[ownedbycats]

I tried test version.

1. Noticed in Statistics dialog, hovering over graph shows '0' for pagecount.

2. Can't get the edit to work. For the single-record fic, I created a record, set it to one day before I completed and percent 99%. Pressing apply, it just... disappeared?

[thiago.eec]

Originally Posted by ownedbycats

I tried test version.

1. Noticed in Statistics dialog, hovering over graph shows '0' for pagecount.

Can you attatch a screenshot?

Originally Posted by ownedbycats

2. Can't get the edit to work. For the single-record fic, I created a record, set it to one day before I completed and percent 99%. Pressing apply, it just... disappeared?

I guess I wasn't clear enough. It's not finished yet, so it does nothing at the moment. It just displays the records.

Maybe I shouldn't send it like that. But only after I finished adding the new option I remembered I had started working on the new records dialog.

Tomorrow I'll send the working version.

[ownedbycats]

Include cursor:

[thiago.eec]

Originally Posted by ownedbycats

Include cursor:
Attachment 218123

That's really strange. I can't reproduce. It works fine here. Anyways, try the new test version below. It's fully functional now.

[thiago.eec]

New test version:

1) Now we have a records editor. You can add, remove and edit your records (except for the last one, matching your calibre library info). It works for any year. When you change the progress, the page count is calculated automatically.

2) Added a new option to the config: you can choose to show read books or read pages in the statistics dialog. The other info will be available hovering the bars.

[thiago.eec]

- Bug fix: sort the records dict treating the idx as integer, instead of text, avoiding the wrong order like '0, 1, 10 ,11, 2, 3,...9'

If you test this version, please leave your feedback. Does the page count show when hovering? Does the records editing work perfectly? Any other suggestions?