-
Notifications
You must be signed in to change notification settings - Fork 679
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
chat: push to record #1435
chat: push to record #1435
Conversation
e330fea
to
e88196b
Compare
e88196b
to
b90a2eb
Compare
231d627
to
9d07612
Compare
I've tested this and it works great! One thing that is really impressive is how quick OpenAI can convert the audio to text. The response is nearly instantaneous |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've asked for a couple of non-functional changes but after that we can merge it, thanks!
9d07612
to
151d18c
Compare
Co-authored-by: Randy Mackay <rmackay9@yahoo.com>
151d18c
to
d9ee4d9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
This PR adds a push to record feature i.e. it will record the audio on mouse left key press down and stop the recording on key release.