This repository was archived by the owner on Sep 30, 2024. It is now read-only.
Azure OpenAI - experimental autocomplete support #56063
Merged
Merged
Conversation
eseliger
previously approved these changes
Aug 21, 2023
eseliger
left a comment
Member
There was a problem hiding this comment.
nice! would be cool to see that be backported to OAI at some point as well :)
Co-authored-by: Erik Seliger <erikseliger@me.com>
948732c to
fbc3f8b
Compare
eseliger
approved these changes
Aug 31, 2023
Contributor
Author
|
Thanks @eseliger once we prove this out for Azure I'll bring the changes across for OAI + gateway. |
Contributor
|
The backport to To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-5.1 5.1
# Navigate to the new working tree
cd .worktrees/backport-5.1
# Create a new branch
git switch --create backport-56063-to-5.1
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 888be7bacdde329f653ae7c592826602466e4279
# Push it to GitHub
git push --set-upstream origin backport-56063-to-5.1
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-5.1Then, create a pull request where the |
github-actions Bot
pushed a commit
that referenced
this pull request
Aug 31, 2023
* spike: Make completions work with Azure * remove response printing * lint fixes * Update internal/completions/client/azureopenai/openai.go Co-authored-by: Erik Seliger <erikseliger@me.com> * update azure autocomplete to look for prompt in first message * clean up some comments --------- Co-authored-by: Erik Seliger <erikseliger@me.com> (cherry picked from commit 888be7b)
chwarwick
referenced
this pull request
in sourcegraph/cody-public-snapshot
Aug 31, 2023
Adds a new openai provider that routes requests though the sourcegraph instance. Currently this reuses the prompt logic from the unstabl-azure-openai provider. This should enable working though the sg backend and then any prompt updates can be handled separately. ## Test plan ran locally with https://github.com/sourcegraph/sourcegraph/pull/56063 <!-- Required. See https://docs.sourcegraph.com/dev/background-information/testing_principles. -->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds autocomplete support for Azure OpenAI using the same mechanism that is used by the fireworks provider. It assumes the prompt is the one and only message.
Test plan
Tested against https://github.com/sourcegraph/cody/pull/875
Configure VS Code extension to use
unstable-openaiConfigured local instance to use
azure-openaiwith gpt turbo 3.5 for completionsEnsure autocomplete & chat continue to work