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

docs:Nvidia Riva Runnables Documentation #18237

Merged
merged 22 commits into from
Mar 28, 2024

Conversation

haydeniw
Copy link
Contributor

  • Description: Documents how to use the Riva runnables to add streamed automatic-speech-recognition (ASR) and text-to-speech (TTS) to chains.
  • Issue: None
  • Dependencies: None
  • Twitter handle: @HaydenWolff1

Copy link

vercel bot commented Feb 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 28, 2024 11:34pm

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. Ɑ: Runnables Related to Runnables 🔌: nvidia Related to nvidia integrations 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Feb 28, 2024
Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure we want to add a new documentation section for utilities. usually utilities are only for things that are used by tools, so they show up in tools

@haydeniw
Copy link
Contributor Author

haydeniw commented Mar 4, 2024

@hwchase17 so it sounds like it should go in tools documentation?

Copy link

vercel bot commented Mar 4, 2024

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

Copy link
Collaborator

@ccurme ccurme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! I think tools is a reasonable place for this (although it's not strictly a tool, all the other text-to-speech implementations are documented in tools).

https://python.langchain.com/docs/integrations/tools

I would recommend re-titling the notebook to "NVIDIA Riva" or similar, as the current title is long and may show up as such in the sidebar of the docs.

@baskaryan baskaryan enabled auto-merge (squash) March 28, 2024 23:25
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Mar 28, 2024
@baskaryan baskaryan merged commit 85deee5 into langchain-ai:master Mar 28, 2024
11 checks passed
gkorland pushed a commit to FalkorDB/langchain that referenced this pull request Mar 30, 2024
- **Description:** Documents how to use the Riva runnables to add
streamed automatic-speech-recognition (ASR) and text-to-speech (TTS) to
chains.
  - **Issue:** None
  - **Dependencies:** None
  - **Twitter handle:** @HaydenWolff1

---------

Co-authored-by: Hayden Wolff <[email protected]>
Co-authored-by: Hayden Wolff <[email protected]>
Co-authored-by: Bagatur <[email protected]>
hinthornw pushed a commit that referenced this pull request Apr 26, 2024
- **Description:** Documents how to use the Riva runnables to add
streamed automatic-speech-recognition (ASR) and text-to-speech (TTS) to
chains.
  - **Issue:** None
  - **Dependencies:** None
  - **Twitter handle:** @HaydenWolff1

---------

Co-authored-by: Hayden Wolff <[email protected]>
Co-authored-by: Hayden Wolff <[email protected]>
Co-authored-by: Bagatur <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder lgtm PR looks good. Use to confirm that a PR is ready for merging. 🔌: nvidia Related to nvidia integrations Ɑ: Runnables Related to Runnables size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants