Added TruGen Avatar Plugin. #4430
Conversation
|
tinalenguyen
left a comment
tinalenguyen
left a comment
There was a problem hiding this comment.
hi, thank you for submitting this PR! i left a few comments, but i couldn't test it out. i hit the max retries and kept receiving this error: Error creating voice to video session: concurrent session limit reached
let me know what may be causing this issue and when i could try again, thanks in advance!
| if i < self._conn_options.max_retry - 1: | ||
| await asyncio.sleep(self._conn_options.retry_interval) | ||
|
|
||
| raise APIConnectionError("Max retry exhaused; Unable to start TruGen.AI Avatar Session.") |
There was a problem hiding this comment.
| raise APIConnectionError("Max retry exhaused; Unable to start TruGen.AI Avatar Session.") | |
| raise APIConnectionError("Max retries exhausted; Unable to start TruGen.AI Avatar Session.") |
| def __init__( | ||
| self, | ||
| *, | ||
| avatar_id: NotGivenOr[str | None] = NOT_GIVEN, |
There was a problem hiding this comment.
could you set a default for avatar_id? maybe a variable DEFAULT_AVATAR_ID
| livekit_api_key = livekit_api_key or (os.getenv("LIVEKIT_API_KEY") or NOT_GIVEN) | ||
| livekit_api_secret = livekit_api_secret or (os.getenv("LIVEKIT_API_SECRET") or NOT_GIVEN) | ||
| if not livekit_url or not livekit_api_key or not livekit_api_secret: | ||
| raise Exception( |
There was a problem hiding this comment.
could you create and use a TrugenException class? this would match the patterns of our avatar plugins
- Added default avatar id to be used when nothing is passed. - Changed API Connection Error phrasing.
|
Hi @tinalenguyen, Cheers. |
3 participants
Authored by: Hari Govind [email protected]