private-gpt icon indicating copy to clipboard operation
private-gpt copied to clipboard

Add support for Azure OpenAI

Open olederle opened this issue 1 year ago • 2 comments

Allows to use Azure OpenAI as LLM and embedding model.

olederle avatar Mar 10 '24 16:03 olederle

Published docs preview URL: https://privategpt-preview-2070757f-a2eb-4a3f-9ba7-cf21ab21280c.docs.buildwithfern.com

github-actions[bot] avatar Mar 10 '24 16:03 github-actions[bot]

@olederle please run make check and make test locally. Apparently there are some issues with make check that are preventing the merge.

imartinez avatar Mar 11 '24 21:03 imartinez

hey guys just a heads up in the doc [fern/docs/pages/manual/llms.mdx] you guys wrote api_version: <api_version> # The API version to use. Default is "2023_05_15" when i tried it it gave me a 404 but worked with dashes instead of underscores 2023-05-15

SeifBach avatar Mar 15 '24 12:03 SeifBach

Published docs preview URL: https://privategpt-preview-31cfc7df-c2f0-4abf-b2ce-51708d09c395.docs.buildwithfern.com

github-actions[bot] avatar Mar 15 '24 14:03 github-actions[bot]

Published docs preview URL: https://privategpt-preview-bd8e80a7-86cd-4bbc-988f-be7da3cc9e2d.docs.buildwithfern.com

github-actions[bot] avatar Mar 15 '24 14:03 github-actions[bot]

Published docs preview URL: https://privategpt-preview-a3fbc74d-159d-471a-813a-61686c78614b.docs.buildwithfern.com

github-actions[bot] avatar Mar 15 '24 14:03 github-actions[bot]

hey guys just a heads up in the doc [fern/docs/pages/manual/llms.mdx] you guys wrote api_version: <api_version> # The API version to use. Default is "2023_05_15" when i tried it it gave me a 404 but worked with dashes instead of underscores 2023-05-15

That is right. For my use case I have used a preview api_version but want to have a stable version as default. But I introduced a typo with this. This is fixed now.

olederle avatar Mar 15 '24 14:03 olederle

@olederle please run make check and make test locally. Apparently there are some issues with make check that are preventing the merge.

@imartinez This should be fixed now and I have also updated the docs.

olederle avatar Mar 15 '24 14:03 olederle