-
Notifications
You must be signed in to change notification settings - Fork 2.9k
feat: add autocomplete prompt template #5176
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
feat: add autocomplete prompt template #5176
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -122,6 +122,7 @@ export class CompletionProvider { | |||
|
|||
private async _getAutocompleteOptions() { | |||
const { config } = await this.configHandler.loadConfig(); | |||
// console.log('debug1 the config was', config?.selectedModelByRole.autocomplete, config.) |
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.
// console.log('debug1 the config was', config?.selectedModelByRole.autocomplete, config.) |
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.
removed
@@ -181,6 +186,8 @@ export class CompletionProvider { | |||
helper, | |||
}); | |||
|
|||
console.log('debug1 the prompt was->', prompt) |
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.
console.log('debug1 the prompt was->', prompt) |
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.
removed
|
||
Example: | ||
|
||
```yaml |
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 is missing an actual example of the prompt templating
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.
added
@@ -566,7 +566,7 @@ export interface LLMOptions { | |||
completionOptions?: CompletionOptions; | |||
requestOptions?: RequestOptions; | |||
template?: TemplateType; | |||
promptTemplates?: Record<string, PromptTemplate>; | |||
promptTemplates?: Partial<Record<keyof PromptTemplates, PromptTemplate>>; |
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.
Nice 👌
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.
Just a few console logs to remove/docs feedback, thanks for adding this! Also looks like a merge conflict popped up.
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.
thanks for making the adjustments!
Description
promptTemplates.autocomplete
now works for autocomplete. we used to have this feature in config.json (tabAutocompleteOptions
) and was missing in config.yamlreference discussion: https://discord.com/channels/1108621136150929458/1359484060371783741/1360375811332505730
Checklist
Screenshots
Testing instructions
.continue
folder