Skip to content

Client abstraction over API resources (prototype) #2

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

Draft
johanste wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

Client abstraction over API resources (prototype) #2

Changes from 3 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
6922ae8
Client abstraction over API resources (prototype)
johanste Jun 9, 2023
e1c33e6
Added some async endpoints
johanste Jun 12, 2023
ac61e45
Added image APIs
johanste Jun 13, 2023
b894768
Allow the use of `model` as a synonym for `deployment_id` when using …
johanste Jun 13, 2023
b4fdcd6
Formatteding client.py (black)
johanste Jun 13, 2023
a4f45ef
Added basic tests, more validation for client
johanste Jun 14, 2023
49c7c8b
Update openai/client.py
johanste Jun 15, 2023
bd4960c
Update openai/client.py
johanste Jun 15, 2023
91e9b58
Review feedback
johanste Jun 17, 2023
f992de1
Merge branch 'client' of github.com:johanste/openai-python into client
johanste Jun 17, 2023
7a31602
Fix tests
johanste Jun 17, 2023
e3b59de
adding docstrings initial
kristapratico Jun 20, 2023
24dad1d
Add live tests (#4)
kristapratico Jun 22, 2023
8bb7d61
Add edit, audio, and moderation APIs to client abstraction (#5)
kristapratico Jun 22, 2023
214242c
Merge branch 'client' into docstrings
kristapratico Jun 22, 2023
917b97e
Better test fixtures
johanste Jun 23, 2023
e1edec4
more docstrings
kristapratico Jun 28, 2023
48adec5
docstrings + type hints + remove prompt edits (deprecated)
kristapratico Jul 19, 2023
e82ed27
fix find/replace mistake
kristapratico Jul 19, 2023
0cfaaff
add missing tests + adjust moderation kwargs passed
kristapratico Jul 24, 2023
d9b43f5
Merge branch 'openai:main' into client
johanste Jul 25, 2023
5809b39
Merge pull request #6 from johanste/docstrings
kristapratico Jul 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions openai/client.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,11 @@ def __init__(
):
"""Create a new OpenAI client.

:param: api_base: The base URL for the API. If not specified, based on ~opeanai.api_base
:param: auth: The authentication method or key to use. If the string value "azuredefault" is passed, it will use ~azure.identity.DefaultAzureCredential
:param: api_version: The API version to use. If not specified, based on ~openai.api_version or ~openai.client.LATEST_AZURE_API_VERSION.
:param backend: One of 'azure' or 'openai'. If not specified, inferred from the auth method or ~openai.api_type
:keyword str api_base: The base URL for the API. If not specified, based on ~opeanai.api_base
:keyword auth: The authentication method or key to use. If the string value "azuredefault" is passed, it will use ~azure.identity.DefaultAzureCredential
:paramtype auth: str or ~openai.client.ApiKeyAuth or ~openai.client.AzureTokenAuth
:keyword str api_version: The API version to use. If not specified, based on ~openai.api_version or ~openai.client.LATEST_AZURE_API_VERSION.
:keyword str backend: One of 'azure' or 'openai'. If not specified, inferred from the auth method or ~openai.api_type
Comment thread
kristapratico marked this conversation as resolved.
"""

#
Expand All @@ -94,7 +95,7 @@ def __init__(

# Pick up api type from parameter or environment
self.backend = backend or (
"azure" if openai.api_type in ("azure", "azure_ad") or isinstance(auth, AzureTokenAuth) else "openai"
"azure" if openai.api_type in ("azure", "azure_ad", "azuread") or isinstance(auth, AzureTokenAuth) else "openai"
)

if self.backend == "azure":
Expand Down