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.

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

Add turbo model #33

Merged
merged 21 commits into from
Oct 8, 2024
Merged

Add turbo model #33

merged 21 commits into from
Oct 8, 2024

Conversation

Jiltseb
Copy link
Collaborator

@Jiltseb Jiltseb commented Oct 8, 2024

This PR has following changes:

  1. Adds the latest turbo model from openAI.
  2. Cleans commit history (happened via contributor timestamp discrepancy).
  3. limit pytorch version to cudnn8 for pip install (torch 2.4.0 and later comes with cudnn9 which is not supported yet in Ctranslate2 models).

MahmoudAshraf97 and others added 20 commits October 2, 2024 07:39
@MahmoudAshraf97 @Jiltseb
.
40e81ef
@MahmoudAshraf97 @Jiltseb
remove duplicate detect_language function
0d82795
@MahmoudAshraf97 @Jiltseb
fix conflicts with systran master
f9b19cb
@jordimas @Jiltseb
Filter out non_speech_tokens in suppressed tokens (SYSTRAN#898)
5e4da4b 
* Filter out non_speech_tokens in suppressed tokens
@MahmoudAshraf97 @Jiltseb
.
bedd61a
@MahmoudAshraf97 @Jiltseb
rename chunk_size to chunk_length for consistency
1058f21
@MahmoudAshraf97 @Jiltseb
review comments
5a7c5a2
@MahmoudAshraf97 @Jiltseb
.
8355ee9
@MahmoudAshraf97 @Jiltseb
fixing docstring
3ee8fa6
@MahmoudAshraf97 @Jiltseb
fix usage with english-only models
64b09b7
@Jiltseb
added licensing comments inthe doc and the code
ea2ba53
@Jiltseb
added formatting checks
9614868
@Jiltseb
update license info
1c22ff3
@Jiltseb @hargunmujral @MahmoudAshraf97
New PR for Faster Whisper: Batching Support, Speed Boosts, and Qualit…
5b2cc66 
…y Enhancements (SYSTRAN#856)

Batching Support, Speed Boosts, and Quality Enhancements

---------

Co-authored-by: Hargun Mujral <83234565+hargunmujral@users.noreply.github.com>
Co-authored-by: MahmoudAshraf97 <hassouna97.ma@gmail.com>
@zh-plus @Jiltseb
Make vad-related parameters configurable for batched inference. (SYST…
6b90b77 
…RAN#923)
@MahmoudAshraf97 @Jiltseb
Remove the usage of transformers.pipeline from `BatchedInferencePip…
c338880 
…eline` and fix word timestamps for batched inference (SYSTRAN#921)

* fix word timestamps for batched inference

* remove hf pipeline
@Jiltseb
handle case when audio is empty, e.g. due to VAD filtering all segments
08ac161
@Jiltseb
formatting checks
a344126
@Jiltseb
add turbo model
0829f60
@Jiltseb
limit pytorch version to cudnn8 for pip install
8186b7e
@Jiltseb Jiltseb requested a review from HRashidi October 8, 2024 09:41
HRashidi
HRashidi approved these changes Oct 8, 2024
View reviewed changes
Copy link

@HRashidi HRashidi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏽

@Jiltseb Jiltseb merged commit 4195aeb into master Oct 8, 2024
@Jiltseb Jiltseb deleted the add_turbo_model branch October 24, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HRashidi HRashidi HRashidi 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.
5 participants
@Jiltseb @HRashidi @MahmoudAshraf97 @jordimas @zh-plus