tensorzero
Add tests with `tensorzero::tags` to `openai-go`
See the tests for openai-python and openai-node that use tensorzero::tags and create a similar test for openai-go.
https://github.com/tensorzero/tensorzero/blob/main/clients/openai-python/tests/test_openai_compatibility.py
https://github.com/tensorzero/tensorzero/blob/main/clients/openai-node/tests/openai-compatibility.test.ts
Hi @anesmeftah - thank you for the interest, as you're a first-time contributor, please submit a PR directly!
Hi @HJStaiff - thank you for your interest! We'll look forward to a PR.
@GabrielBianconi Can issues be assigned once a person expresses interest? That way it's easier to filter on unassigned issues. Perhaps un-assign if the assigned person doesn't provide a PR within X days?
Hi @michaldorsett - we used to do that in the past, but the majority of people who requested to be assigned never sent a PR, and it was tough to keep track of "stale assignments." For now we're not explicitly assigning issues to people who haven't contributed, though PRs are welcomed!
Hi! I’ve started looking into this issue. I’ll post updates here as I make progress.
Hi! I'm currently assigned to this issue, and after reviewing the repo, I found that the compatibility test for openai-go (including tensorzero::tags) already exists in openai_compatibility_test.go.
Just wanted to confirm if this issue is still open for additional work or if it's already been completed. Happy to help further if needed!
@GabrielBianconi The test is in: clients/openai-go/tests/openai_compatibility_test.go , There are several test cases that use the tensorzero:: format for example, in the TestBasicInference function, call:
req.WithExtraFields(map[string]any{ "tensorzero::episode_id": episodeID.String(), })
There are also other tests using tensorzero::function_name::basic_test, tensorzero::variant_name, and similar tags. Let me know if I misunderstood the issue or missed something.
We need a test that covers tensorzero::tags specifically (literally), see the other files for Python and Node for example.
I’ll go ahead and add a new test that specifically uses tensorzero::tags, similar to what’s in the Python and Node tests.Appreciate the guidance I’ll follow up shortly with the changes!