Revert "Changed environment variable names OPENAI_API_BASE and OPENAI…

[larinam]

…_API_VERSION to AZURE_OPENAI_API_BASE and AZURE_OPENAI_API_VERSION"

This reverts commit ce8b29e.

Please see the documentation for langchain.embeddings.openai.OpenAIEmbeddings to get the point of the environment varibles naming. These names should not be changed.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs-gpt	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 5, 2023 0:11am

[larinam]

Hi @dartpain ,
Could you review this one? If additional explanation is required, I can provide it.
The thing with naming is that in the langchain package, it is expected to have particular names for the environment variables. When the change which I suggest to revert here was introduced, it led to the broken Azure support in DocsGPT. The revert of the commit will recover functionality which was in place before the change introduced by the commit, which is suggested to be reverted in this pull request.
Thank you for your time reviewing these changes and maintaining this project!

[dartpain]

Thank you for you explanation, it seems to be working fine for me on the dev environment and quickstart one

#276 I just want to be careful not to break it again.