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

Add Hamming-weighted average to DelayedAggregation #18

Merged
merged 10 commits into from
Dec 21, 2021
Merged

Conversation

juanmc2005
Copy link
Owner

@juanmc2005 juanmc2005 commented Dec 17, 2021

Issue

This PR addresses issue #13, #17 and partly #15.

Changelog

  • Decouple aggregation strategy from DelayedAggregation
  • Add HammingWeightedAverageStrategy, AverageStrategy and FirstOnlyStrategy
  • Rename strategy "any" to "first", which is more descriptive
  • Replace UnreliableFileAudioSource in demo.py that was causing issues with ReliableFileAudioSource
  • Fix bugs in ReliableFileAudioSource
  • Add is_regular property to AudioSource to indicate whether a source should be regularized or not, for example in OnlineSpeakerDiarization pipeline
  • Add optional duration property to AudioSource in order to output last non-aggregated predictions
  • Replace buffer_with_count with custom buffer_slide operator
  • Fix: initial predictions were being ignored because the number of buffers to aggregate didn't match the expected amount
  • Fix: last latency - step seconds were being ignored in streamed files
  • Make OutputBuilder draw as soon as the first buffer and adapt the visualization extents dynamically
  • Add optional stream_end parameter to DelayedAggregation

@juanmc2005 juanmc2005 added bug Something isn't working feature New feature or request labels Dec 17, 2021
@juanmc2005 juanmc2005 self-assigned this Dec 17, 2021
@juanmc2005 juanmc2005 merged commit 2cb1554 into develop Dec 21, 2021
@juanmc2005 juanmc2005 deleted the feat/hamming branch December 21, 2021 13:26
@juanmc2005 juanmc2005 mentioned this pull request Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant