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

Command: always test the prompt #383

Merged
merged 1 commit into from
Jan 7, 2023
Merged

Command: always test the prompt #383

merged 1 commit into from
Jan 7, 2023

Conversation

dnhkng
Copy link
Contributor

@dnhkng dnhkng commented Jan 7, 2023

This adds a new mode for the Command example.

Adding a custom prompt parameter with -p or --prompt activates this mode, eg.:

./command --prompt "OK whisper, please"

Command will only output valid commands, which must always be prepended by the given promptsl and have a word count > 0. Otherwise, nothing is output.

This is designed to match the behaviour of devices like Alexa and Siri.

@ggerganov
Copy link
Owner

Haha - very nice work! I tried it and it works perfect!
Will merge this and do some clean-up and refactoring in the command.cpp on master

@ggerganov ggerganov merged commit 9c4a152 into ggerganov:master Jan 7, 2023
abitofevrything pushed a commit to abitofevrything/whisper.cpp that referenced this pull request Jan 8, 2023
abitofevrything pushed a commit to abitofevrything/whisper.cpp that referenced this pull request Jan 8, 2023
rock3125 pushed a commit to rock3125/whisper.cpp that referenced this pull request Feb 21, 2023
rock3125 pushed a commit to rock3125/whisper.cpp that referenced this pull request Feb 21, 2023
anandijain pushed a commit to anandijain/whisper.cpp that referenced this pull request Apr 28, 2023
anandijain pushed a commit to anandijain/whisper.cpp that referenced this pull request Apr 28, 2023
jacobwu-b pushed a commit to jacobwu-b/Transcriptify-by-whisper.cpp that referenced this pull request Oct 24, 2023
jacobwu-b pushed a commit to jacobwu-b/Transcriptify-by-whisper.cpp that referenced this pull request Oct 24, 2023
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