Skip to content
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

Refactor audio message to Dialogue Utterance #384

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

derkmed
Copy link
Contributor

@derkmed derkmed commented Feb 2, 2024

This is the first of several mini-PRs that will cover the changes proposed in #378

We should remove the previously used messages upon confirmation of working implementation.

@derkmed
Copy link
Contributor Author

derkmed commented Feb 2, 2024

@Purg @hdefazio @joshanderson-kw This should be in a reviewable state. It refactors all nodes to use a new general purpose message that allows on-the-fly re-ordering or removal of audio nodes depending on compute-latency constraints.

The next PRs will attempt to bring ASR + VAD locally, add visual question answering, and add @ovenmitt 's demo changes.
Please review as soon as possible to unblock these upcoming changes.

@hdefazio
Copy link
Contributor

@derkmed can you rebase on the latest master?

@Purg
Copy link
Member

Purg commented May 20, 2024

@derkmed Is this relevant anymore with the more recent PR of yours (#418) having landed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants