search

SO-Close-Vote-Reviewers / SOCVR-Chatbot

Chatbot (lovingly named Closey) for the SO Close Vote Reviewers Chat Room
GNU General Public License v2.0
16 stars 2 forks source link

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

Open adeak opened 8 years ago

adeak commented 8 years ago

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 commented 8 years ago

Possibly related #149.

adeak commented 8 years ago

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 commented 8 years ago

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