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

Upgrade to v1.7.0 #71

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Upgrade to v1.7.0 #71

wants to merge 42 commits into from

Conversation

jwijffels
Copy link
Contributor

@jwijffels jwijffels commented Oct 6, 2024

Linked to

  • upgrade to whisper.cpp 1.5.5 #64

  • whisper turbo #67

  • Latest release #61

  • to_timestamp: compile common.cpp

  • timestamp_to_sample: compile common.cpp

  • replace_all: not needed, removed

  • whisper_params: changed to the newest version with new params
    new params:

  • audio_ctx = 0

  • grammar

  • grammar_rule

  • grammar_penalty=100

  • grammar_parsed

  • temperature=0

  • temperature_inc=0.2

  • suppress_regex (regex to suppress tokens)

  • dtw: "" dynamic time warping

  • estimate_diarization_speaker: ok, kept as is

  • whisper_print_progress_callback: ok, keep the old behaviour, nothing changed

  • whisper_print_segment_callback: ok, keep the old behaviour, nothing changed

  • WhisperModel: ok, kept as is

  • whisper_load_model: ok, kept as is

  • whisper_encode: handled changes internally (but not in the whisper_encode API yet)

  • audio_ctx = 0

  • temperature=0

  • temperature_inc=0.2

  • suppress_regex (regex to suppress tokens)
    Left out changes regarding grammar

  • grammar

  • grammar_rule

  • grammar_penalty=100

  • grammar_parsed
    Not yet included dynamic time warping (dtw)

  • whisper_print_benchmark: ok, kept as is

  • whisper_language_info: ok, moved to rcpp_whisper_utils.cpp

Looks ok already, still need to handle all the changes to Makevars

@jwijffels jwijffels mentioned this pull request Nov 4, 2024
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.

1 participant