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

Use shuffled LibriSpeech cuts instead #1450

Merged
merged 2 commits into from
Jan 8, 2024
Merged

Use shuffled LibriSpeech cuts instead #1450

merged 2 commits into from
Jan 8, 2024

Conversation

JinZr
Copy link
Collaborator

@JinZr JinZr commented Jan 5, 2024

No description provided.

@JinZr JinZr marked this pull request as ready for review January 5, 2024 11:44
@JinZr JinZr marked this pull request as draft January 5, 2024 11:45
@JinZr
Copy link
Collaborator Author

JinZr commented Jan 8, 2024

Sys. ID test-clean test-other comment
1. Baseline System 2.27 5.10 --epoch 30 --avg 9
2. Baseline System 2.23 4.96 --epoch 40 --avg 16
3. Shuffled Cuts. 2.25 5.15 --epoch 30 --avg 10
4. Shuffled Cuts. 2.24 5.02 --epoch 40 --avg 17

All systems were decoded using greedy search with the default decoding configuration.

@JinZr JinZr marked this pull request as ready for review January 8, 2024 04:41
@danpovey
Copy link
Collaborator

danpovey commented Jan 8, 2024

OK.
I'd rather leave the old code in comments though, in case anyone is using an older setup, and to make them aware of the possibility of doing this kind of thing.

@JinZr
Copy link
Collaborator Author

JinZr commented Jan 8, 2024

noted, the latest commit should do that

@JinZr JinZr merged commit 5445ea6 into k2-fsa:master Jan 8, 2024
54 checks passed
@JinZr JinZr deleted the fix/shuf branch January 8, 2024 07:09
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.

3 participants