-
Notifications
You must be signed in to change notification settings - Fork 972
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
FEAT: Complete implementation of GGML_OP_CONV_1D
#523
Conversation
Thanks! I just tested this by merging Whisper uses the ggml/examples/whisper/whisper.cpp Lines 1542 to 1561 in 7ddb66d
Repro:
|
If you rebase to latest
It will run the CI locally and at the end of the run is the Whisper test.
|
@ggerganov Thanks for pushing a way to test conv 1d. I took the code from
|
Works for me now! @ggerganov
|
Awesome - will take a look in the next days |
@ggerganov Can somebody have a look please? I need it to complete |
@PABannier Yes, sorry for the delay - was travelling for the past week. I'm back now and will catch up with everything today and tomorrow |
Currently, the 1d convolution is only implemented for half padding and stride 1 and 2. Yet, the 1d convolution is a crucial operation, needed for instance in bark.cpp and encodec.cpp .
This PR completes the implementation of the 1d convolution (for f32 and f16 src types). It also updates the computation of the size needed for the work buffer.