-
Notifications
You must be signed in to change notification settings - Fork 773
enhance OpenAi conversation component documentation for Azure OpenAi #4750
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
enhance OpenAi conversation component documentation for Azure OpenAi #4750
Conversation
Signed-off-by: fabistb <[email protected]>
Signed-off-by: fabistb <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fabistb - See my comment
| `cacheTTL` | N | A time-to-live value for a prompt cache to expire. Uses Golang duration format. | `10m` | | ||
| `apiType` | N | Specifies the API provider type. Required when using a provider that does not follow the default OpenAI API endpoint conventions. | `azure` | | ||
| `apiVersion`| N | The API version to use. Required when the `apiType` is set to `azure`. | `2025-01-01-preview` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fabistb - Is 2025-01-01-preview
the only current valid value or are there others? If so, is there a link to Azure documentation for this list that we can reference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some preview apis as listed here:
https://learn.microsoft.com/en-us/azure/ai-foundry/openai/api-version-lifecycle?tabs=key#preview-api-releases
The newest version is: 2025-04-01-preview
We could add the latest stable version as example value, but some of the newer models won't work then.
If that works for you I would however add link above to the related links
section
…ecycle Signed-off-by: fabistb <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Enhance the documentation for the OpenAI conversation API component.
Functionality to also utilize Azure OpenAi is added.
This includes two new optional metadata properties:
Also adds some description how the different metadata properties must be configured that Azure OpenAi works.
Related PR:
dapr/components-contrib#3918
Issue reference
#4749