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 recipe for the yes_no dataset. #380

Merged
merged 2 commits into from Aug 21, 2021
Merged

Conversation

csukuangfj
Copy link
Contributor

No description provided.

@csukuangfj
Copy link
Contributor Author

See k2-fsa/icefall#16

@pzelasko
Copy link
Collaborator

Thanks! Could you add an entry in the corpora list in docs/corpus.rst?

@csukuangfj
Copy link
Contributor Author

Thanks! Could you add an entry in the corpora list in docs/corpus.rst?

Ok. Just added an entry to it and fixed several typos.

@pzelasko
Copy link
Collaborator

Ok. Just added an entry to it and fixed several typos.

Nice, thanks! BTW are you adopting black for k2? I might reformat whole Lhotse codebase with it at some point…

@pzelasko pzelasko merged commit 2a1410b into lhotse-speech:master Aug 21, 2021
@csukuangfj csukuangfj deleted the yesno branch August 21, 2021 23:00
@csukuangfj
Copy link
Contributor Author

Ok. Just added an entry to it and fixed several typos.

Nice, thanks! BTW are you adopting black for k2? I might reformat whole Lhotse codebase with it at some point…

I am using black for icefall, but yapf for k2.
Maybe we should also use black to reformat the files in k2 at some point.

@pzelasko pzelasko added this to the v0.8 milestone Aug 26, 2021
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.

None yet

2 participants