Skip to content

Commit

Permalink
fix double argument issue
Browse files Browse the repository at this point in the history
  • Loading branch information
Pwuts committed May 31, 2024
1 parent 58c755a commit cd9a5d7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion forge/forge/llm/providers/_openai_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -340,11 +340,11 @@ async def _create_chat_completion(
int: Number of prompt tokens used
int: Number of completion tokens used
"""
completion_kwargs["model"] = completion_kwargs.get("model") or model

Check warning on line 343 in forge/forge/llm/providers/_openai_base.py

View check run for this annotation

Codecov / codecov/patch

forge/forge/llm/providers/_openai_base.py#L343

Added line #L343 was not covered by tests

@self._retry_api_request
async def _create_chat_completion_with_retry() -> ChatCompletion:
return await self._client.chat.completions.create(

Check warning on line 347 in forge/forge/llm/providers/_openai_base.py

View check run for this annotation

Codecov / codecov/patch

forge/forge/llm/providers/_openai_base.py#L345-L347

Added lines #L345 - L347 were not covered by tests
model=model,
**completion_kwargs, # type: ignore
)

Expand Down

0 comments on commit cd9a5d7

Please sign in to comment.