encodec.cpp icon indicating copy to clipboard operation
encodec.cpp copied to clipboard
verified publisher icon PABannier

Steps needed to support the 48khz model

Open kalradivyanshu opened this issue 2 years ago • 1 comments

Hey, Thankyou for all your work, I was wondering can this be extended to support the 48khz model? What all will that require, is it simple? Would love to contribute if someone can provide some guidance.

Thanks!

kalradivyanshu avatar Nov 21 '23 05:11 kalradivyanshu

Hey @kalradivyanshu ! Thank you for your interest in the project.

I don't think it would be difficult. I made sure the codebase remains modular and that hyperparameters are not hardcoded, so it should come down to:

  1. Parsing correctly the weights of the 48Khz model
  2. Ensure the weights are correctly loaded in encodec.cpp
  3. Make sure the forward pass is identical

You can open a PR and I'd be happy to guide you in the process.

PABannier avatar Nov 23 '23 10:11 PABannier