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

Azure OpenAI Service Provider with configurable model deployment #336

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

Conversation

Adeas
Copy link

@Adeas Adeas commented Aug 27, 2024

This PR introduces support for Azure OpenAI as a service provider within the application. It ensures that any missing models from the user's deployed models are gracefully skipped, allowing the application to continue functioning without interruption.

Key updates include:

  • Configuration: Added Azure OpenAI configuration parameters to config.toml (example provided in sample.config.toml). Users must specify their Azure OpenAI endpoint, model deployment details, and API key.

  • Model Loading: The implementation attempts to load available Azure OpenAI models, logging warnings for any missing or failed models while still proceeding with the available ones.

  • Security Considerations: The API key is visible in the Perplexica UI/Backend (e.g., in the UI Settings and when accessing http://127.0.0.1:3001/api/models). Users should be cautious and ensure that access to the UI/Backend is secured.

Please ensure that your Azure OpenAI endpoint and deployment details are correctly configured, and be aware of the potential visibility of sensitive keys in the application interface.

@ksingh-scogo
Copy link

@ItzCrazyKns Do you plan to merge this , this feature would definitely help in adoption of this project.

@lokdao
Copy link

lokdao commented Oct 2, 2024

@ItzCrazyKns It would be awesome to have this feature :)

@ItzCrazyKns
Copy link
Owner

Hi @Adeas, thanks for the PR! All the changes looks good to me, the only thing I'd recommend adding is the input in the settings dialog for configuring Azure models. Rest of the changes LGTM.

@xeniode
Copy link

xeniode commented Oct 3, 2024

Woot! I was doing this myself. Glad this is happening. Thanks @Adeas & @ItzCrazyKns

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

Successfully merging this pull request may close these issues.

5 participants