add comments to agent side and inference side fallback adapters#5654
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| """Agent Fallback Adapter for LLM. Manages multiple STT instances with automatic fallback | ||
| when the primary provider fails. | ||
| """ |
There was a problem hiding this comment.
🟡 LLM FallbackAdapter docstring incorrectly says "STT instances" instead of "LLM instances"
The newly added class docstring for the LLM FallbackAdapter at livekit-agents/livekit/agents/llm/fallback_adapter.py:37 says "Manages multiple STT instances" but this class manages LLM instances (as seen from self._llm_instances and the llm: list[LLM] parameter). This is a copy-paste error from the STT fallback adapter docstring.
| """Agent Fallback Adapter for LLM. Manages multiple STT instances with automatic fallback | |
| when the primary provider fails. | |
| """ | |
| """Agent Fallback Adapter for LLM. Manages multiple LLM instances with automatic fallback | |
| when the primary provider fails. | |
| """ |
Was this helpful? React with 👍 or 👎 to provide feedback.
| """Agent Fallback Adapter for TTS. Manages multiple STT instances with automatic fallback | ||
| when the primary provider fails. |
There was a problem hiding this comment.
🟡 TTS FallbackAdapter docstring incorrectly says "STT instances" instead of "TTS instances"
The updated class docstring for the TTS FallbackAdapter at livekit-agents/livekit/agents/tts/fallback_adapter.py:49 says "Manages multiple STT instances" but this class manages TTS instances (as seen from self._tts_instances and the tts: list[TTS] parameter). This is a copy-paste error from the STT fallback adapter docstring.
| """Agent Fallback Adapter for TTS. Manages multiple STT instances with automatic fallback | |
| when the primary provider fails. | |
| """Agent Fallback Adapter for TTS. Manages multiple TTS instances with automatic fallback | |
| when the primary provider fails. |
Was this helpful? React with 👍 or 👎 to provide feedback.
|
This is an automated Claude Code Routine created by @toubatbrian. Right now it is in experimentation stage. The automation will start porting this PR into agents-js automatically. The Python PR adds searchable "Agent Fallback Adapter" / "Inference Fallback Adapter" prefixes to the docstrings on the agent-side fallback adapters (LLM/STT/TTS) and the inference-side
Generated by Claude Code |
3 participants
We're updating the docs to describe how users can write model fallbacks on the LiveKit Agent side and on the LiveKit Inference side. We would like the comments to make it easy to find the code for the agent side fallback and the inference side fallback.