Skip to content

Phoneme Detection and Classifier Model Codes - #238

Open
Anirudh Bindiganavale Harish (AnirudhBHarish) wants to merge 8 commits into
microsoft:masterfrom
AnirudhBHarish:kws_phoneme
Open

Phoneme Detection and Classifier Model Codes#238
Anirudh Bindiganavale Harish (AnirudhBHarish) wants to merge 8 commits into
microsoft:masterfrom
AnirudhBHarish:kws_phoneme

Conversation

@AnirudhBHarish

No description provided.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor Comments.

Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/README.md Outdated
Comment thread applications/KWS_Phoneme/auxiliary_files/README.md Outdated
Comment thread applications/KWS_Phoneme/auxiliary_files/README.md Outdated
Comment thread applications/KWS_Phoneme/train_classifier.py Outdated
Comment thread applications/KWS_Phoneme/train_phoneme.py Outdated
@AnirudhBHarish

Copy link
Copy Markdown
Author

I apologize for the delay. I have incorporated all the typographical and punctuational changes in the new commit.

Comment thread applications/KWS_Phoneme/auxiliary_files/README.md Outdated
Comment thread applications/KWS_Phoneme/auxiliary_files/README.md Outdated
Comment thread applications/KWS_Phoneme/kwscnn.py Outdated
@ShikharJ

Copy link
Copy Markdown
Contributor

Harsha Simhadri (@harsha-simhadri) I'm not the right person to review the Python scripts, so if Sachin and Anirudh feel they are fine, then I guess we should get them merged.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Minor comment, rest everything looks good.

Comment thread applications/KWS_Phoneme/data_pipe.py Outdated
Comment thread applications/KWS_Phoneme/kwscnn.py Outdated
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.

2 participants