Skip to content
This repository has been archived by the owner on Apr 16, 2023. It is now read-only.

Closey reports inconsistent review times in manual vs automatic reports #154

Open
adeak opened this issue Mar 17, 2016 · 3 comments
Open
Assignees
Labels

Comments

@adeak
Copy link

adeak commented Mar 17, 2016

Issue:
Message by Closey (~9 PM UTC) in response to reviews today:

You've reviewed 53 posts today (of which 3 were audits). The time between your first and last review today was 21 hours, 19 minutes, and 22 seconds, averaging to a review every 24 minutes and 8 seconds.

5 minutes later, automatic message on reaching review limit:

You've reviewed 60 posts today (of which 6 were audits), thanks! The time between your first and last review today was 45 minutes and 29 seconds, averaging to a review every 45 seconds.

Even after this point, reviews today returns the former (21 hour) time window:

You've reviewed 59 posts today (of which 5 were audits). The time between your first and last review today was 21 hours, 22 minutes, and 39 seconds, averaging to a review every 21 minutes and 44 seconds.

And now I see that I also lost an audit (the whole review event) in this last report. Probably deleted since? The latest report above was produced almost an hour later.

Background:
I reviewed a few items after midnight, then stopped for the most of the day without interacting with Closey. I carried on in the evening, leading to the above messages. The 21 hour time window in the first message is clearly due to my post-midnight reviews, and the second (shorter time window) might be consistent with my evening contiguous reviewing session.

@ArcticEcho ArcticEcho self-assigned this Mar 17, 2016
@ArcticEcho
Copy link
Contributor

Possibly related #149.

@adeak
Copy link
Author

adeak commented Mar 18, 2016

I've confirmed that the shorter time span reported automatically corresponds to the time elapsed between the "I'll update your session record" message in the evening, and the corresponding message. The manual messages, on the other hand, use the first reviews of the given day, in accordance with the wording of the report.

@ArcticEcho
Copy link
Contributor

Ok, thanks for the extra info. I'll aim to get this fixed by next weekend.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

2 participants