feat: Add v1 OpenAI endpoint support and remove legacy completions API#2560
Conversation
|
Hey @ranadeepsingh 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2560 +/- ##
==========================================
+ Coverage 84.69% 84.78% +0.08%
==========================================
Files 335 334 -1
Lines 17753 17756 +3
Branches 1615 1634 +19
==========================================
+ Hits 15036 15054 +18
+ Misses 2717 2702 -15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Copilot encountered an error: Your billing is not configured or you have Copilot licenses from multiple standalone organizations or enterprises. To use premium requests, select a billing entity via the GitHub site, under Settings > Copilot > Features.
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
BrendanWalsh
left a comment
There was a problem hiding this comment.
Looks good from my review.
Related Issues/PRs
#xxx
What changes are proposed in this pull request?
This PR updates SynapseML OpenAI integrations to support OpenAI v1-compatible base URLs such as:
https://<resource_name>.services.ai.azure.com/openai/v1Changes include:
OpenAIPrompt.modelfield and omits theapi-versionquery parameter.api-version.apiVersion, sinceapiVersionis ignored for v1-style endpoints.OpenAICompletionAPI implementation, response schema, and tests.OpenAIChatCompletion,OpenAIPrompt, andOpenAIResponses.How is this patch tested?
Tested with the
synapsemlconda environment:conda run -n synapseml black --check --extend-exclude 'docs/' .conda run -n synapseml sbt "cognitive / Test / compile"conda run -n synapseml sbt "cognitive / Test / testOnly com.microsoft.azure.synapse.ml.services.openai.OpenAIV1EndpointSuite"conda run -n synapseml sbt scalastyle test:scalastylegit diff --checkNote:
sbt scalafmtCheckAllwas attempted but this build does not define that sbt task.Does this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
This adds support for OpenAI v1-compatible base URLs. Current OpenAI notebooks/docs were updated to remove retired
OpenAICompletionexamples and describe supported chat, responses, and embedding APIs.