Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Daily classifications count is lost when you go to Talk #5081

Closed
eatyourgreens opened this issue Jul 28, 2023 · 2 comments · Fixed by #5082
Closed

Daily classifications count is lost when you go to Talk #5081

eatyourgreens opened this issue Jul 28, 2023 · 2 comments · Fixed by #5082
Labels
bug Something isn't working degraded-ux UX is degraded

Comments

@eatyourgreens
Copy link
Contributor

eatyourgreens commented Jul 28, 2023

Your daily session count is stored in volatile storage here, which means that it isn't saved in the panoptes-user snapshot when you leave the Classify page. As a result, it starts again from 0 each time that you return.

One subtle side effect of this bug is that sessionCount % 5 will always be 0 here, meaning that you'll be nagged about workflow promotion on every subject, if you classify using Done & Talk rather than Done.

get sessionCountIsDivisibleByFive() {
return self.sessionCount % 5 === 0
},

sessionCount is also used to calculate your total classification count here, leading to the impression that classifications have been lost when the volatile state resets to 0.

get totalClassificationCount() {
const activityCount = self.projectPreferences?.activity_count || 0
return activityCount + self.sessionCount
}

@eatyourgreens eatyourgreens added bug Something isn't working degraded-ux UX is degraded labels Jul 28, 2023
@eatyourgreens
Copy link
Contributor Author

This bug can give the impression that classifications have been lost and might be the bug discussed here: bug concerning vanishing classifications statistics / resetting total classifications counter (Zooniverse Talk.)

@eatyourgreens
Copy link
Contributor Author

Also worth noting that sessionCount increments from zero as long as a tab is open, so isn’t a daily count either. Fixed in #5082 by counting daily classifications separately from classifications for the current browser session.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working degraded-ux UX is degraded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant