-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add Azure Responses Content Image sample with image url and image base64 #44818
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
base: main
Are you sure you want to change the base?
Add Azure Responses Content Image sample with image url and image base64 #44818
Conversation
Thank you for your contribution @waltervvb! We will review the pull request and get back to you soon. |
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.
This looks really good. I think it has a couple of formatting issues here and there, but it's almost ready! Thank you.
...-openai/src/samples/java/com/azure/ai/openai/responses/AzureResponsesContentImageSample.java
Outdated
Show resolved
Hide resolved
.../azure-ai-openai/src/samples/java/com/azure/ai/openai/responses/ChainingResponsesSample.java
Outdated
Show resolved
Hide resolved
sdk/openai/azure-ai-openai/src/test/java/com/azure/ai/openai/responses/AzureResponsesTest.java
Outdated
Show resolved
Hide resolved
...enai/azure-ai-openai/src/test/java/com/azure/ai/openai/responses/AzureResponsesTestBase.java
Outdated
Show resolved
Hide resolved
...enai/azure-ai-openai/src/test/java/com/azure/ai/openai/responses/AzureResponsesTestBase.java
Outdated
Show resolved
Hide resolved
ResponsesResponse response = client.createResponse(request); | ||
assertImageResponseForAzure(response); | ||
} | ||
|
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.
Both tests are great! Once we've locked down on the implementation we will need to write the async version and the NonAzure
versions too.
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.
NonAzure and Async Versions added
…nstead of passing the full response .
… only for api-version 2025-03-01-preview and later
API change check API changes are not detected in this pull request. |
@@ -31,7 +31,7 @@ static Stream<Arguments> getTestParametersResponses() { | |||
} | |||
|
|||
private static boolean supportsResponses(AzureResponsesServiceVersion serviceVersion) { | |||
return AzureResponsesServiceVersion.V2024_12_01_PREVIEW.equals(serviceVersion); | |||
return AzureResponsesServiceVersion.V2025_03_01_PREVIEW.equals(serviceVersion); |
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.
I am currious, was this change necessary?
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.
I was receiving this message "Azure OpenAI Responses API is enabled only for api-version 2025-03-01-preview and later"
|
@microsoft-github-policy-service agree company="Aditi Consulting LLC" |
Description
Add two samples for the Responses interface to generate model responses using image inputs.