-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
tdrz and coreml support? #1088
Comments
cc @akashmjn, who proposed the original (awesome) PR. This is important for speed improvements on iPhones/Mac. |
Was able to workaround this by adding https://github.com/akashmjn/tinydiarize/blob/886a8d3523c5bf6e8bfc57f3441f1ce6f4345ad4/whisper/__init__.py#L23 to the whisper package installed in my miniconda environment
Probably a better way, but it worked heh... |
@akeybl do you have this model uploaded somewhere? Perhaps we could add it to the same folder on HuggingFace where @akashmjn already saved the non CoreML version? https://huggingface.co/akashmjn/tinydiarize-whisper.cpp |
Hey @akeybl thanks for the cc! i was on break for a bit, hence the delay. Looks like i missed the coreml conversion in this PR #1001 I'll take a look and fix this later today (both conversion from pytorch checkpoint, and directly supporting Excited to see someone give this a spin on an iPhone! |
@akashmjn would it also be possible to get |
1). Regarding a finetuned 2). I just looked into This can be done either by
3). Regarding I don't actually have the right Mac/iPhone hardware atm to test the CoreML stuff so let me know how it goes. |
@akashmjn, would a base.en-tdrz have the same issue? |
The link has expired. Does anyone have the pt file hosted somewhere else? https://github.com/akashmjn/tinydiarize/blob/886a8d3523c5bf6e8bfc57f3441f1ce6f4345ad4/whisper/__init__.py#L23 |
Interested in this as well! |
If any of you have worked with the tdrz model last year, could you please check your |
I tried the ggml model to load, don't understand how to load this model |
Is anyone else experiencing issues with this? All I see when running the small model using the "hack" above is
|
Setting up a new macbook pro, m2, added coreml, works great! Except with new trdz feature.
running
./models/generate-coreml-model.sh small.en-tdrz
is missing from conversion script list of options.The text was updated successfully, but these errors were encountered: