Add required env parameters for azure usage #700
Merged
+12
−4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nponeccop
previously approved these changes
Apr 10, 2023
|
@Rokniel There are conflicts again due to the merging of other PRs |
|
Fixed the conflicts. Changed the naming to follow the new master branch naming changes |
nponeccop
approved these changes
Apr 11, 2023
1 task
|
Also fixed in #802 seems to be the same implementation. |
sweetlilmre
added a commit
to sweetlilmre/Auto-GPT
that referenced
this pull request
Apr 12, 2023
updated README.md to explain new config added Azure yaml loader to config class centralized model retrieval into config class this commit effectively combines and replaces Significant-Gravitas#700 and Significant-Gravitas#580
5 tasks
nponeccop
approved these changes
Apr 12, 2023
richbeales
approved these changes
Apr 12, 2023
EdwardYuan
pushed a commit
to EdwardYuan/Auto-GPT
that referenced
this pull request
Apr 13, 2023
updated README.md to explain new config added Azure yaml loader to config class centralized model retrieval into config class this commit effectively combines and replaces Significant-Gravitas#700 and Significant-Gravitas#580
korla-plankton
pushed a commit
to korla-plankton/Auto-GPT
that referenced
this pull request
Apr 14, 2023
updated README.md to explain new config added Azure yaml loader to config class centralized model retrieval into config class this commit effectively combines and replaces Significant-Gravitas#700 and Significant-Gravitas#580
SquareandCompass
pushed a commit
to SquareandCompass/Auto-GPT
that referenced
this pull request
Oct 21, 2023
…/github_actions/actions/setup-python-4 Bump actions/setup-python from 2 to 4
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
Related to issue #657. Azure openAI uses multiple separate deployments for chat, embeddings and general "AI". These deployments must be specifically called for functionality.
Changes
Two new parameters have been added to the env file. One for embedding deployment and one for chat. The parameters are accessible from config.py within the use_azure=true block. llm_utils.py and memory/base.py have been changed to use these parameters if use_azure=true
Documentation
Additional parameters are described in readme.md under the Installation header within the If you want to use Azure -block.
Test Plan
Tested general functionality against our azure openai deployments. With the parameters in place, it works appears to work identically to a openAI API config
PR Quality Checklist