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

chore: Added ru locale, added voximplant adapter, groq provider, switched to whisper in deepgram (multilang) #304

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

iowathe3rd
Copy link

bunch of chore code, not serius

@prateeksachan prateeksachan self-assigned this Jul 3, 2024
@@ -42,7 +43,7 @@
'anyscale': LiteLLM,
'custom': OpenAiLLM,
'ola': OpenAiLLM,
'groq': LiteLLM,
'groq': GroqLLM,
Copy link
Member

Choose a reason for hiding this comment

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

any specific reason of not using litellm for groq?

Copy link
Author

Choose a reason for hiding this comment

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

Hi, I'm sorry for the very dirty pr, I'll correct it and write in the comments

Copy link
Author

Choose a reason for hiding this comment

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

any specific reason of not using litellm for groq?

in cases when user need to add more then 1 litellm driven provider,

real case example:
I could be wrong, but in our project, which is a fork of your repository, we need to use both groq and antrophic at the same time, from env variables

@@ -0,0 +1,28 @@
aiobotocore==2.9.0
Copy link
Member

@prateeksachan prateeksachan Jul 3, 2024

Choose a reason for hiding this comment

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

these packages could be installed when installing bolna package in the docker

Copy link
Author

Choose a reason for hiding this comment

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

I almost missed the mark, I'm not that good at python

print(f'websocket_url: {websocket_url}')
print(f'recipient_phone_number: {recipient_phone_number}')

voxapi = VoximplantAPI("credentials/voximplant.json")
Copy link
Member

Choose a reason for hiding this comment

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

can you also please commit a test/temporary sample voximplant json file.

Copy link
Author

Choose a reason for hiding this comment

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

Integration with Voximplant should be two-way, unfortunately there is no built-in ability to stream audio from a websocket, but on the bolna side everything is ok

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

Successfully merging this pull request may close these issues.

None yet

3 participants