search

Canner / WrenAI

🚀 Open-source SQL AI Agent for Text-to-SQL. Make Text2SQL Easy! 🙌
https://getwren.ai/oss
GNU Affero General Public License v3.0
1.73k stars 155 forks source link

Support 3rd party openai api #365

Closed RoacherM closed 3 months ago

RoacherM commented 3 months ago

Now, you can specify the OPENAI_API_BASE in your environment variables to utilize any third-party OPENAI services. This feature allows for greater flexibility and customization in accessing and integrating with various OPENAI API endpoints provided by different service providers. By setting the OPENAI_API_BASE environment variable, you can seamlessly switch between or utilize multiple third-party OPENAI services, enhancing the adaptability and functionality of your applications.

RoacherM commented 3 months ago

@wwwy3y3 Hi, I've submitted a pull request (#365) that adds support for specifying third-party OpenAI API services. This feature allows users to set the OPENAI_API_BASE environment variable to utilize different third-party OpenAI services, enhancing flexibility and customization.

Could you please review and approve this pull request? Thank you for your time and assistance!

cyyeh commented 3 months ago

@RoacherM sure, thanks for your contribution! I'll help you review this PR

wwwy3y3 commented 3 months ago

@RoacherM thanks for your contribution! @cyyeh will follow-up review this PR.

cyyeh commented 3 months ago

@RoacherM Thanks again for your hard work! I've left some comments. Feel free to discuss with me if you found other issues :)

RoacherM commented 3 months ago

Thanks for your review. I've made some modifications as per your suggestions.

cyyeh commented 3 months ago

@RoacherM I've merged your hard work to the main branch. Since now we support adding OPENAI_API_BASE to support other OpenAI api compatible models, I also created a PR for adding the support for embedding models. If you are interested, would you like to test this scenario? I suppose this is more complete :) https://github.com/Canner/WrenAI/pull/376

RoacherM commented 3 months ago

I'd be glad to test the new functionality for supporting embedding models. Thank you for merging my work into the main branch. I will review the code changes and test the embedding model scenario as soon as possible.

On Tue, Jun 4, 2024 at 22:21 Chih-Yu Yeh @.***> wrote:

@RoacherM https://github.com/RoacherM I've merged your hard work to the main branch. Since now we support adding OPENAI_API_BASE to support other OpenAI api compatible models, I also created a PR for adding the support for embedding models. If you are interested, would you like to test this scenario? I suppose this is more complete :) #376 https://github.com/Canner/WrenAI/pull/376

— Reply to this email directly, view it on GitHub https://github.com/Canner/WrenAI/pull/365#issuecomment-2147667030, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH73HHQIIPRVN7R2KFOSSUDZFXEPPAVCNFSM6AAAAABIVUY35GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXGY3DOMBTGA . You are receiving this because you were mentioned.Message ID: @.***>