feat(cohere): trace audio transcription calls#333
Open
Abhijeet Prasad (AbhiPrasad) wants to merge 1 commit intomainfrom
Open
feat(cohere): trace audio transcription calls#333Abhijeet Prasad (AbhiPrasad) wants to merge 1 commit intomainfrom
Abhijeet Prasad (AbhiPrasad) wants to merge 1 commit intomainfrom
Conversation
Instrument `client.audio.transcriptions.create()` (sync and async) on
Cohere's v1 clients so Braintrust captures transcription requests added
in cohere==6.1.0. Spans record the audio file as an `Attachment`, the
transcribed text as output, provider/model/language/temperature as
metadata, and standard timing metrics.
- patchers target `cohere.audio.transcriptions.client.{Async,}TranscriptionsClient.create`
via `target_module`, so the integration degrades gracefully on older
Cohere SDKs that predate the audio surface.
- `wrap_cohere()` walks `client.audio.transcriptions` to instance-patch
the leaf `create` method for per-client tracing.
- `clean_cohere_methods` test fixture now also clears wrapt-forwarded
markers on the restored original function.
Covered by three VCR-backed tests (sync, async, setup) against the real
`v2/audio/transcriptions` endpoint.
Fixes #327
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Instrument
client.audio.transcriptions.create()(sync and async) on Cohere's v1 clients so Braintrust captures transcription requests added in cohere==6.1.0. Spans record the audio file as anAttachment, the transcribed text as output, provider/model/language/temperature as metadata, and standard timing metrics.cohere.audio.transcriptions.client.{Async,}TranscriptionsClient.createviatarget_module, so the integration degrades gracefully on older Cohere SDKs that predate the audio surface.wrap_cohere()walksclient.audio.transcriptionsto instance-patch the leafcreatemethod for per-client tracing.clean_cohere_methodstest fixture now also clears wrapt-forwarded markers on the restored original function.Covered by three VCR-backed tests (sync, async, setup) against the real
v2/audio/transcriptionsendpoint.Fixes #327