Skip to content

ruby : Add no_speech_thold #2641

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.

Sign up for GitHub

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

Jump to bottom
Merged
ggerganov merged 19 commits into from
Dec 18, 2024
Merged

ruby : Add no_speech_thold #2641

ggerganov merged 19 commits into from
Dec 18, 2024

Conversation

@KitaitiMakoto
Copy link
Collaborator

@KitaitiMakoto KitaitiMakoto commented Dec 17, 2024

Hello,

This pull request includes:

Thanks.

@KitaitiMakoto
Remove Whisper::Model.[]
6c780b5
@KitaitiMakoto
Fix Whisper::Model::URI#request
f99bc8a
@KitaitiMakoto
Make Whisper::Context#initialize accept pre-converted model name
241db0c
@KitaitiMakoto
Use downloading pre-converted model feature for testing
cde4b58
@KitaitiMakoto
Update README
b8ff450
@KitaitiMakoto
Remove unnecessary task
6960117
@KitaitiMakoto
Move whisper/model.rb -> whisper/model/uri.rb
71b700b
@KitaitiMakoto
Update document comment of Whisper::Context#initialize
bffe4a1
@KitaitiMakoto
Don't show download progress when not tty
36e84da
@KitaitiMakoto
Pass String to raise
dec133e
@KitaitiMakoto
Use cache model file if download fails
3f628b1
@KitaitiMakoto
Add test for auto download
4f632dd
@KitaitiMakoto
Specify required Ruby version
ede026d
@KitaitiMakoto
Fix a typo
fb4ec92
@KitaitiMakoto
Remove unnecessary flags
b89d397
@KitaitiMakoto
Initialize Whisper::Params#diarize explicitely
a71b8ff
@KitaitiMakoto
Remove redundant code from README for simplicity
218f22e
@KitaitiMakoto
Add Whisper::Params#no_speech_thold attribute
ca11274
@KitaitiMakoto
Add test for Whisper::Params#no_speech_thold
4d9b736
@ggerganov ggerganov merged commit 3a27b2b into ggml-org:master Dec 18, 2024
42 checks passed
@KitaitiMakoto KitaitiMakoto deleted the flexible-model-spec branch December 18, 2024 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ggerganov ggerganov ggerganov approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KitaitiMakoto @ggerganov