Skip to content

fix: stop dataset mismatch alert from flashing during autodetection - #1850

Merged
ivan-aksamentov merged 2 commits into
masterfrom
fix/autodetect-error-blink
Nov 3, 2026
Merged

ivan-aksamentov merged 2 commits into
masterfrom
fix/autodetect-error-blink

Conversation

@ivan-aksamentov

Copy link
Copy Markdown
Member

Note

✨ Claude Opus 5.5 is writing on behalf of Ivan

Change: Mark a dataset autodetection run as done only together with its suggestions [src].

Context: The run became done when the worker finished streaming per-sequence results, but the suggestions arrived from a later worker call. In between, the main page and the dataset page briefly showed the red "Possible dataset mismatch detected" or "No matching datasets" alert, even for matching sequences.

Work items

  • Resolve the autodetection run only after the worker has delivered every result, and terminate the worker after each run [src]
  • Remove commented-out suggestion helpers

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

…detection

- The run state became done when the worker finished streaming per-sequence results, but the best datasets arrived only after a further worker call, so for that interval the suggestion alerts read the run as finished without suggestions and showed the red "No matching datasets" alert
- Commit the best datasets and the done state together, after the worker has delivered every result, so that no component observes a finished run with an incomplete outcome
- Terminate the autodetection Web Worker after each run, because each run spawns a new worker and freeing only its WebAssembly module left the worker alive
@ivan-aksamentov
ivan-aksamentov force-pushed the fix/autodetect-error-blink branch from 1b703ed to c9ea5a1 Compare November 3, 2026 03:00
@ivan-aksamentov
ivan-aksamentov merged commit c9ea5a1 into master Nov 3, 2026
2 checks passed
@ivan-aksamentov
ivan-aksamentov deleted the fix/autodetect-error-blink branch November 3, 2026 03:00
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 this pull request may close these issues.

1 participant