pipecat icon indicating copy to clipboard operation
pipecat copied to clipboard

LLM user frame processor with tests

Open EdgarMMProsper opened this issue 1 year ago • 1 comments

Proposed solution for this issue: https://github.com/pipecat-ai/pipecat/issues/441 We try to improve the processing of frames from the user, since they often arrive out of order. We also tackle the case when the VAD does not activate but we receive transcriptions. We follow the principle: no user input should be lost. We use interruptions for making sure the retrigger of the LLM does not create odd responses

EdgarMMProsper avatar Sep 10 '24 08:09 EdgarMMProsper

Thank you so much @EdgarMMProsper ! We are releasing Pipecat today but I'll make sure to review this so it goes into the next release.

aconchillo avatar Oct 02 '24 20:10 aconchillo

@aconchillo I have been using this code since the hackathon, I found that it solved the problem of having some tokens not added to the LLM context. Would be amazing to see it merged at some point if you feel code is good :)

skandere avatar Nov 06 '24 20:11 skandere

@aconchillo I have been using this code since the hackathon, I found that it solved the problem of having some tokens not added to the LLM context. Would be amazing to see it merged at some point if you feel code is good :)

Cool! This validation actually helps a lot. This is definitely on the top of my list and I'm sorry I haven't looked at it and merged it before. 😞 This is coming next week for sure!

aconchillo avatar Nov 07 '24 22:11 aconchillo

Hey! I confirm that we have not touched this code in our internal codebase since this PR and we are still consistently happy about it. At some point we had to disable the interim transcriptions, because Deepgram was sending them offtime (like sending Interim Final TheSameInterimAgain)

EdgarMMProsper avatar Nov 07 '24 23:11 EdgarMMProsper

Any idea when this'll be able to be merged?

Vortigern-source avatar Nov 09 '24 22:11 Vortigern-source

Hi! I'm moving this PR here #703. I want to move some things around. I'm keeping the origin commit and will work on top of it. I will release Pipecat 0.0.48 tomorror (Sunday) and quick smaller release with this PR after that.

aconchillo avatar Nov 09 '24 22:11 aconchillo