Skip to content
Snippets Groups Projects
Verified Commit 572c36c7 authored by Sofiane Lasri's avatar Sofiane Lasri
Browse files

feat: remove redundant parameters from AI provider configuration

- Removed 'temperature', 'top_p', 'frequency_penalty', and 'presence_penalty' parameters from the AI provider configuration in AiProviderService.php.
- Cleaned up logging of the request details before making the API call.
parent eb2ce456
No related branches found
No related tags found
No related merge requests found
Pipeline #1039 passed
......@@ -75,11 +75,7 @@ public function promptWithPictures(string $systemRole, string $prompt, UploadedP
],
],
],
'temperature' => 1,
'max_tokens' => config('ai-provider.providers.'.$selectedProvider.'.max-tokens'),
'top_p' => 1,
'frequency_penalty' => 0,
'presence_penalty' => 0,
'response_format' => [
'type' => 'json_object',
],
......@@ -114,11 +110,7 @@ public function prompt(string $systemRole, string $prompt): array
],
],
],
'temperature' => 1,
'max_tokens' => config('ai-provider.providers.'.$selectedProvider.'.max-tokens'),
'top_p' => 1,
'frequency_penalty' => 0,
'presence_penalty' => 0,
'response_format' => [
'type' => 'json_object',
],
......@@ -138,11 +130,6 @@ private function callApi(string $url, array $requestBody): array
{
$selectedProvider = config('ai-provider.selected-provider');
Log::info('Calling AI provider API', [
'url' => $url,
'request_body' => $requestBody,
]);
try {
$response = Http::withHeaders([
'Authorization' => 'Bearer '.config('ai-provider.providers.'.$selectedProvider.'.api-key'),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment