Making pipecat work with Krisp

[filipi87]

Creating Krisp filter for audio processor.

[aconchillo]

@filipi87 We will be using an input transport filter instead: #611. I think I will merge that now, so you can use that instead.

[aconchillo]

@filipi87 We will be using an input transport filter instead: #611. I think I will merge that now, so you can use that instead.

This is now merged.

[filipi87]

Easy way to choose which model we wish to use.

[aconchillo]

I don't see any new environment variables...

[filipi87]

We are using it here inside krisp_filter.py, in case a model path is not provided.

[filipi87]

While the library has not been published yet, you can test it by creating a local distribution using the PR and then installing it.

[filipi87]

Hi @aconchillo , we are going to be able to publish the pipecat-ai-krisp, but without including the Krisp SDK, so we don't go against any policy from Krisp. Thanks again for all your suggestions. 🙂

So, in summary, the pipecat-ai-krisp module that we are going to publish will have the source code for the wrapper, along with all the scripts and everything else that we need to build it.

At install time, we are going to receive the path to the SDK as an environment variable, and at that point, we will build and install it.

So it will work like this:

export KRISP_SDK_PATH=/PATH/TO/YOUR/KRISP/SDK
pip install pipecat-ai-krisp

//or while we don't have published it yet, you can test with the local version:
pip install ../pipecat-krisp/dist/pipecat_krisp-0.1.0.tar.gz

[aconchillo]

I would say something like ## Reduce background noise with audio filters (just to make it more attractive) and then a subsection for Krisp.

[aconchillo]

Any chance the project can be called pipecat-ai-krisp? Just to match pipecat-ai. There's another pipecat project completely unrelated.

[aconchillo]

The project I mean the pypi package. The repo is fine.

[aconchillo]

Oh, I see it's already pipecat-ai-krisp, then I guess just updating this text. :-D

[filipi87]

Yep, it seems I have missed this text. 😬

[aconchillo]

I'd squash this into a single commit probably, or two one for the filter and one for the example.

[aconchillo]

LGTM! Just a couple of minor things. This is awesome!

[filipi87]

Great. I will address the missing things and squash in a single commit before merging it.
Thank you @aconchillo. 🙂

[markbackman]

Very minor feedback. We already have a 07o foundational example (i.e. 07o-interruptible-assemblyai.py). Can the example be 07p-interruptible-krisp.py?