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

Visualize speaker self-overlaps in the details plot #88

Merged
merged 6 commits into from
Sep 3, 2024

Conversation

thequilo
Copy link
Member

@thequilo thequilo commented Aug 29, 2024

Fixes the (long broken) warning for self-overlap in the UI and splits a speaker column into multiple sub-columns if utterances of the same speaker overlap.

This is an example of the overlap visualization in the details plot
image

This way of visualization is not pretty since the lines connecting matching words overlap, but it shows all relevant information and is much better than the previous behavior.

I also cleaned up the code a little:

  • Removed the partial support for multiple hypothesis in the same visualization in favor of the synchronized side-by-side view
  • The horizontal positions of the utterances are now precomputed
  • The WER displayed in the visualization now matches the WER computed by the CLI tool exactly
  • It is now easier to add assignment algorithms, like DI-cp, in the future
  • Added buttons to quickly jump to the next/prev utterance with self-overlap

@thequilo thequilo merged commit 68b7870 into main Sep 3, 2024
6 checks passed
@thequilo thequilo deleted the viz-warn-self-overlap branch September 3, 2024 11:52
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