Change Batch quickstart to use new client, focus on Python SDK#222
Change Batch quickstart to use new client, focus on Python SDK#222J-Jaywalker wants to merge 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
One thought - out of the scope of this MR - but the |
| print(result.transcript_text) | ||
| await client.close() | ||
|
|
||
| asyncio.run(main()) No newline at end of file |
| AUDIO_FILE= "example.wav" | ||
|
|
||
| config = TranscriptionConfig( | ||
| diarization="speaker" |
There was a problem hiding this comment.
Should this include operating_point = standard?
There was a problem hiding this comment.
I'm open to discuss this.
I'd like to show how quick/easy it is to get started with batch, so if there's a noticeably higher RTF for enhanced than standard then I agree we should put it in. Otherwise maybe we just leave it out to be barebones and reduce mental clutter.
For context I added the transcription config because without it our speaker would be diarized as SPEAKER_UU which doesn't seem intentional(?).
Open to discussion though.
| </Tabs> | ||
|
|
||
|
|
||
| ### Transcript response schema |
There was a problem hiding this comment.
Re our discussion on teams, this is also available on the output page. You could move the example response to the output page too, and have a next steps section which references the output page and some cool features as well?
The current docs point to our old (deprecated) Python client instead of the shiny new one.
I don't think I'll have time to comb through the entire docs and update them, but this change tweaks the Batch API quickstart page to focus more on Python and use code from our updated client.