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

Streaming ww dev #162

Merged
merged 181 commits into from
Sep 9, 2024
Merged

Streaming ww dev #162

merged 181 commits into from
Sep 9, 2024

Conversation

jeremy-syn
Copy link
Contributor

Merging work on the new streaming wakeword benchmark into the master branch. All the changes in streaming_ww_dev benchmark should be under benchmark/training/streaming_wakeword, but there are quite a lot of them.

…where background_volume option was being ignored
…function. Applies to val set now, but still not test set
Replaced flatten with reshape in order to preserve time duration.
Adding empty noise frames and duplicates of the wakeword to
validationset (instead of just training)
number of duplicates and noise level are now command line flags and are
separate for training and validation sets.
Input shape is now (batch, time, 1, features) (None,49,1,40) instead of
just (None,49,40) to avoid the extra expand_dims layer, which caused
problems for QAT.
@jeremy-syn jeremy-syn requested a review from a team as a code owner August 12, 2024 16:05
Copy link

github-actions bot commented Aug 12, 2024

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@Peter-Chang Peter-Chang self-assigned this Sep 9, 2024
Copy link
Contributor

@Peter-Chang Peter-Chang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Peter-Chang Peter-Chang merged commit 0c6a990 into master Sep 9, 2024
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants