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

WIP: openai settings #5792

Merged
merged 2 commits into from
Jun 7, 2023
Merged

WIP: openai settings #5792

merged 2 commits into from
Jun 7, 2023

Conversation

hwchase17
Copy link
Contributor

[] need to test more
[] make sure they arent saved when serializing
[] do for embeddings

@hwchase17 hwchase17 merged commit 3954bcf into master Jun 7, 2023
12 checks passed
@hwchase17 hwchase17 deleted the harrison/openai-settings branch June 7, 2023 04:57
@jigarchhadwa
Copy link

[] need to test more [] make sure they arent saved when serializing [] do for embeddings

Do we know why we are hard coding api_type as azure? We were using the managed identity to authenticate with AzureOpenAI, and for that api_type is "azure_ad"... Our code stopped working because of this change..
openai_api_type: str = "azure"

Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
[] need to test more
[] make sure they arent saved when serializing
[] do for embeddings
This was referenced Jun 25, 2023
kacperlukawski pushed a commit to kacperlukawski/langchain that referenced this pull request Jun 29, 2023
[] need to test more
[] make sure they arent saved when serializing
[] do for embeddings
@anderl80
Copy link

Could this help as well to solve #6202?

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.

None yet

3 participants