Skip to content
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.

Sign up for GitHub

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

refactor(forge/llm): Create BaseOpenAIProvider -> deduplicate GroqProvider & OpenAIProvider #7178

Merged
merged 3 commits into from
Jun 2, 2024

fix double argument issue

cd9a5d7
Select commit
Loading
Failed to load commit list.
Merged

refactor(forge/llm): Create BaseOpenAIProvider -> deduplicate GroqProvider & OpenAIProvider #7178

fix double argument issue
cd9a5d7
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded May 31, 2024 in 0s

34.66% of diff hit (target 33.75%)

View this Pull Request on Codecov

34.66% of diff hit (target 33.75%)

Annotations

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L67 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L70 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L72 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L76 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L79 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L81 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L88 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L95 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L98 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L114 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L129 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L131 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L134 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L147 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L164 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

forge/forge/llm/providers/_openai_base.py#L176-L180 

Added lines #L176 - L180 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L184 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

forge/forge/llm/providers/_openai_base.py#L188-L189 

Added lines #L188 - L189 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L191 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L193 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L196 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L200 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L202 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added line #L207 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

forge/forge/llm/providers/_openai_base.py#L209-L212 

Added lines #L209 - L212 were not covered by tests
View more details on Codecov