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.

Sign up for GitHub

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

Jump to bottom

Use it with Azure OpenAI #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Use it with Azure OpenAI #17

wants to merge 2 commits into from
+2 −1

Conversation

yakigac
Copy link

@yakigac yakigac commented Oct 6, 2024

This pull request updates the RealtimeAPI class in lib/api.js to support other URLs, such as Azure OpenAI.

Changes:

  • Updated WebSocket URL to be dynamic (this.url). (lib/api.js)
  • Added api-key header for Azure OpenAI. (lib/api.js)

@svapnil
Copy link

svapnil commented Oct 9, 2024

is Realtime live on Azure now?

@yakigac
Copy link
Author

yakigac commented Oct 11, 2024

@svapnil yes.

https://learn.microsoft.com/ja-jp/azure/ai-services/openai/realtime-audio-quickstart?pivots=programming-language-ai-studio

@transitive-bullshit transitive-bullshit mentioned this pull request Oct 16, 2024
Open
threepointone
threepointone reviewed Oct 21, 2024
View reviewed changes
lib/api.js Outdated Show resolved Hide resolved
@yakigac @threepointone
Update lib/api.js
4249b7d 
Co-authored-by: Sunil Pai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@threepointone threepointone threepointone left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yakigac @svapnil @threepointone