Skip to content

Commit dde067f

Browse files
authored
Fix missing embedding endpoint in patch_openai_client, unify test (tensorzero#3193)
* added missing endpoint * set up OpenAI python tests to work with patched client * removed all references to clients/openai-python * added pyright ignore in test fixture
1 parent f106e31 commit dde067f

File tree

15 files changed

+178
-1052
lines changed

15 files changed

+178
-1052
lines changed

.github/workflows/general.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -305,18 +305,6 @@ jobs:
305305
run: |
306306
uv run stubtest tensorzero.tensorzero
307307
308-
- name: "Python: OpenAI Client: Install dependencies"
309-
working-directory: clients/openai-python
310-
run: |
311-
uv venv
312-
uv pip sync requirements.txt
313-
314-
- name: "Python: OpenAI Client: pyright"
315-
working-directory: clients/openai-python
316-
run: |
317-
uv pip install pyright==1.1.394
318-
uv run pyright
319-
320308
- name: "Node.js: Run prettier"
321309
run: pnpm --filter=openai-node run format
322310

.github/workflows/merge-queue.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -238,18 +238,6 @@ jobs:
238238
fi
239239
exit 1
240240
241-
- name: "Python: OpenAI Client: Install dependencies"
242-
working-directory: clients/openai-python
243-
run: |
244-
uv venv
245-
uv pip sync requirements.txt
246-
247-
- name: "Python: OpenAI Client: pytest"
248-
working-directory: clients/openai-python
249-
timeout-minutes: 5
250-
run: |
251-
uv run pytest --verbose
252-
253241
- name: "Node.js: OpenAI Client: test"
254242
working-directory: clients/openai-node
255243
run: |

.pre-commit-config.yaml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -174,13 +174,6 @@ repos:
174174
- repo: https://github.com/RobertCraigie/pyright-python
175175
rev: v1.1.391
176176
hooks:
177-
- id: pyright
178-
name: "pyright: clients/openai-python"
179-
files: ^clients/openai-python/
180-
args: ["--project", "clients/openai-python/pyrightconfig.json"]
181-
additional_dependencies:
182-
- openai
183-
184177
- id: pyright
185178
name: "pyright: recipes"
186179
files: ^recipes/

clients/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# TensorZero Gateway Clients
22

3-
The TensorZero Gateway can be used with the **TensorZero Python client** (see `./python`), with **OpenAI clients (see `./openai-python`, `./openai-node`, and `./openai-go` for examples)**, or via its **HTTP API in any programming language**.
3+
The TensorZero Gateway can be used with the **TensorZero Python client** (see `./python`), with **OpenAI clients (see `./python/tests`, `./openai-node`, and `./openai-go` for examples)**, or via its **HTTP API in any programming language**.
44

55
Refer to [TensorZero Gateway Clients](https://www.tensorzero.com/docs/gateway/clients) in our documentation for usage details and more information.

clients/openai-python/pyproject.toml

Lines changed: 0 additions & 12 deletions
This file was deleted.

clients/openai-python/pyrightconfig.json

Lines changed: 0 additions & 3 deletions
This file was deleted.

clients/openai-python/requirements.txt

Lines changed: 0 additions & 335 deletions
This file was deleted.

clients/openai-python/tests/conftest.py

Lines changed: 0 additions & 22 deletions
This file was deleted.

clients/openai-python/uv.lock

Lines changed: 0 additions & 525 deletions
This file was deleted.

clients/python/DEVELOPER_NOTES.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@ uv run python
2626

2727
Integration tests can be run with `./test.sh` (this requires the same setup as `cargo test-e2e` - see `CONTRIBUTING.md`)
2828

29+
This also runs all tests for OpenAI compatibility in Python.
30+
2931
## Naming
3032

3133
There are several different names in use in this client:

0 commit comments

Comments
 (0)