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

towards audio #1048

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

towards audio #1048

wants to merge 1 commit into from

Conversation

pelikhan
Copy link
Member

@pelikhan pelikhan commented Jan 24, 2025


Sure, here's a summary of the key changes in the GIT_DIFF:

  • Added support for audio input (input_audio) to the public API. This allows models like Ayudiuo to be used with different types of input.

  • Enhanced PromptNodeRender by adding audios, keeping a list of resolved audio outputs.

  • Updated renderPromptNode to process and handle audio nodes, moving similar code blocks to improve readability.

  • Fixed a bug in createChatGenerationContext that does not support the generation of images or audio (using throw new NotSupportedError) since these models are not fully supported yet.

AI-generated content by pr-describe may be incorrect

Copy link

LGTM 🚀

The changes introduce support for audio content in prompts and render them accordingly. This is well-structured and comprehensive. The modifications to the TypeScript files are clear and functional, ensuring that the new features are correctly implemented.

I reviewed the diff and didn't find any immediate issues or concerns. The addition of audio handling appears to be a valuable feature and is well-implemented across all relevant files.

If you have further tests or documentation that need to be updated to fully support this new functionality, consider adding them as follow-up work. Otherwise, these changes are ready for merging. 📗

AI-generated content by pr-review may be incorrect

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.

1 participant