-
Notifications
You must be signed in to change notification settings - Fork 775
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
Add support for moonshine ASR models #990
Comments
Hi, Evan from Useful Sensors/Moonshine here. Just letting you know this is on our radar. We're working on Transformers support right now, and we (internally) have our current ONNX models running in the browser with |
@evmaki Not related to this issue, but I am also eagerly waiting for the ability to finetune for supporting a new language. |
@evmaki Great to hear! I have been following the ONNX support and it looks like a great start! One issue is that you currently export two versions of the decoder (w/ and w/o PKVs), leading to weight duplication (more of a problem when running in the browser since we load the decoder twice). We were able to solve this in Optimum by adding an If node to the graph and then choosing which path to take based on whether the past key values are provided. See here for an example. And here is the code used to merge the two decoders. I was experimenting with your codebase to pass zero-sized tensors as input, but I get gibberish output. Either way, once we have |
Model description
Please add support for moonshine ASR models. The recent github commit adds support for onnx(python), so I guess porting to js won't take much effort. However, there is no mention about transformers usage.
This model is quite good for in-browser usage scenario since it is quite small and claims to use RAM proportional to length of audio.
Prerequisites
Additional information
No response
Your contribution
None
The text was updated successfully, but these errors were encountered: