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

Classifier: Long delays (up to 10s) after pressing Done #6698

Open
eatyourgreens opened this issue Feb 17, 2025 · 3 comments · May be fixed by #6699
Open

Classifier: Long delays (up to 10s) after pressing Done #6698

eatyourgreens opened this issue Feb 17, 2025 · 3 comments · May be fixed by #6699

Comments

@eatyourgreens
Copy link
Contributor

eatyourgreens commented Feb 17, 2025

Volunteers on Daily Minor Planet are reporting delays of up to 10s after pressing the Done button.
https://www.zooniverse.org/talk/17/3531984?comment=5898899

The delay seems to be in submitClassification here, which waits for the classification to save before showing you a new subject. It would be faster if it behaved like PFE and saved the classification in the background, while you classify the next subject.

function * submitClassification (classification) {
// Service worker isn't working right now, so let's use the fallback queue for all browsers
try {
yield self.classificationQueue.add(classification)
self.loadingState = asyncStates.posting
} catch (error) {
console.error(error)
self.loadingState = asyncStates.error
}
}

Here, I've captured a couple of classifications, in the console, which took ~5s and ~4s respectively.

Screenshot of the console log for a couple of Daily Minor Planet classifications which took 4.79s and 3.98s.

@eatyourgreens
Copy link
Contributor Author

eatyourgreens commented Feb 17, 2025

One caveat: if you press Done & Talk, then you do want the classifier to pause until the classification is saved. Otherwise, the browser tab will cancel the network request (and any running JS) when the page unloads (#4767.)

@eatyourgreens eatyourgreens linked a pull request Feb 17, 2025 that will close this issue
12 tasks
@eatyourgreens
Copy link
Contributor Author

Here's a link to the full discussion on DMP Talk. This problem has been going on for at least a few weeks now.
https://www.zooniverse.org/projects/fulsdavid/the-daily-minor-planet/talk/5639/3570312

@eatyourgreens
Copy link
Contributor Author

Delays of 4s to 10s for each classification mean the FEM frontend is, unfortunately, almost impossible to use at the moment.

@eatyourgreens eatyourgreens changed the title Classifier: Long delays after pressing Done Classifier: Long delays (up to 10s) after pressing Done Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant