Skip to content

Added speaker identification, punctuation, and recording controls - #3

Open
neeharika2802 wants to merge 1 commit into
oscurprof:mainfrom
neeharika2802:feature/speaker-identification-punctuation
Open

Added speaker identification, punctuation, and recording controls#3
neeharika2802 wants to merge 1 commit into
oscurprof:mainfrom
neeharika2802:feature/speaker-identification-punctuation

Conversation

@neeharika2802

Copy link
Copy Markdown

No description provided.

@JSGRD22

JSGRD22 commented Aug 10, 2026

Copy link
Copy Markdown

This PR seems to do more than it claims, like removing all comments in the code, rewriting the README into that of a "fork", adding unwarranted settings.json and replacing the locally-run argostranslate with the online and privacy-unfriendly GoogleTranslator. Please review with care.

@neeharika2802

Copy link
Copy Markdown
Author

Thank you for the review! I apologize for the extra noise in this PR. Some of my local editor settings and formatting got accidentally caught in the commit. I will revert the README, restore the comments, remove the settings.json, and revert the translation engine back to argostranslate. I'll push a clean update shortly.

@oscurprof

Copy link
Copy Markdown
Owner

I just saw some people started working on this repository, which indeed seems very cool to me. I left working on it sometime ago, due to mainly professional life. I might get some free time in couple of days and will merge the pull requests (if feasible)

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