Skip to content

Commit 4dd39b7

Browse files
committed
convert the time budget integer to a real number.
1 parent db82927 commit 4dd39b7

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

Extension/src/LanguageServer/copilotCompletionContextProvider.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ export class CopilotCompletionContextProvider implements ContextResolver<CodeSni
7272
private static readonly providerId = 'ms-vscode.cpptools';
7373
private readonly completionContextCache: Map<string, CacheEntry> = new Map();
7474
private static readonly defaultCppDocumentSelector: DocumentSelector = [{ language: 'cpp' }, { language: 'c' }, { language: 'cuda-cpp' }];
75-
private static readonly defaultTimeBudgetFactor: number = 0.5;
75+
// A percentage expressed as an integer number, i.e. 50 means 50%.
76+
private static readonly defaultTimeBudgetFactor: number = 50;
7677
private static readonly defaultMaxCaretDistance = 4096;
7778
private completionContextCancellation = new vscode.CancellationTokenSource();
7879
private contextProviderDisposable: vscode.Disposable | undefined;
@@ -177,7 +178,7 @@ export class CopilotCompletionContextProvider implements ContextResolver<CodeSni
177178
private async fetchTimeBudgetFactor(context: ResolveRequest): Promise<number> {
178179
try {
179180
const budgetFactor = context.activeExperiments.get(CopilotCompletionContextProvider.CppCodeSnippetsTimeBudgetFactor);
180-
return (budgetFactor as number) ?? CopilotCompletionContextProvider.defaultTimeBudgetFactor;
181+
return ((budgetFactor as number) ?? CopilotCompletionContextProvider.defaultTimeBudgetFactor) / 100.0;
181182
} catch (e) {
182183
console.warn(`fetchTimeBudgetFactor(): error fetching ${CopilotCompletionContextProvider.CppCodeSnippetsTimeBudgetFactor}, using default: `, e);
183184
return CopilotCompletionContextProvider.defaultTimeBudgetFactor;
@@ -186,8 +187,8 @@ export class CopilotCompletionContextProvider implements ContextResolver<CodeSni
186187

187188
private async fetchMaxDistanceToCaret(context: ResolveRequest): Promise<number> {
188189
try {
189-
const budgetFactor = context.activeExperiments.get(CopilotCompletionContextProvider.CppCodeSnippetsMaxDistanceToCaret);
190-
return (budgetFactor as number) ?? CopilotCompletionContextProvider.defaultMaxCaretDistance;
190+
const maxDistance = context.activeExperiments.get(CopilotCompletionContextProvider.CppCodeSnippetsMaxDistanceToCaret);
191+
return (maxDistance as number) ?? CopilotCompletionContextProvider.defaultMaxCaretDistance;
191192
} catch (e) {
192193
console.warn(`fetchMaxDistanceToCaret(): error fetching ${CopilotCompletionContextProvider.CppCodeSnippetsMaxDistanceToCaret}, using default: `, e);
193194
return CopilotCompletionContextProvider.defaultMaxCaretDistance;

0 commit comments

Comments
 (0)