Skip to content

Commit

Permalink
Autocomplete: Remove outdated embeddings config (#1308)
Browse files Browse the repository at this point in the history
Leftover from removing the embeddings context from Autocomplete

## Test plan

👀 

<!-- Required. See
https://docs.sourcegraph.com/dev/background-information/testing_principles.
-->
  • Loading branch information
philipp-spiess authored Oct 4, 2023
1 parent b487093 commit cf59647
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
8 changes: 1 addition & 7 deletions vscode/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -923,13 +923,7 @@
"llama-code-13b",
"llama-code-13b-instruct"
],
"markdownDescription": "Overwrite the default model used for code autocompletion inference. This is only supported with the `unstable-fireworks` provider"
},
"cody.autocomplete.advanced.embeddings": {
"order": 99,
"type": "boolean",
"default": true,
"markdownDescription": "Enables the use of embeddings as code completions context."
"markdownDescription": "Overwrite the model used for code autocompletion inference. This is only supported with the `unstable-fireworks` provider"
},
"cody.autocomplete.completeSuggestWidgetSelection": {
"type": "boolean",
Expand Down
2 changes: 0 additions & 2 deletions vscode/src/configuration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,6 @@ describe('getConfiguration', () => {
return 'starcoder-32b'
case 'cody.autocomplete.advanced.accessToken':
return 'foobar'
case 'cody.autocomplete.advanced.embeddings':
return false
case 'cody.autocomplete.completeSuggestWidgetSelection':
return false
case 'cody.autocomplete.experimental.syntacticPostProcessing':
Expand Down

0 comments on commit cf59647

Please sign in to comment.