Skip to content

Conversation

zygiert1990
Copy link
Contributor

No description provided.

@zygiert1990 zygiert1990 self-assigned this Mar 7, 2025
Comment on lines 19 to 31
/** Generates audio from the input text.
*
* [[https://platform.openai.com/docs/api-reference/audio/createSpeech]]
*
* @param requestBody
* Request body that will be used to create a speech.
*
* @return
* The audio file content.
*/
def createSpeech(requestBody: SpeechRequestBody): Request[Either[OpenAIException, InputStream]] =
client.createSpeechAsInputStream(requestBody)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure does it make any sense to put here this method, as this is just delegation to client method, which doesn't do any further action.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it doesn't as this extension is about streaming, and there's no streaming here

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(well, no ox-streaming, just regular Java InputStream)

@zygiert1990 zygiert1990 changed the title add create speech method with akka streams implementation extend Audio, Models & Threads API's Mar 7, 2025
@zygiert1990 zygiert1990 changed the title extend Audio, Models & Threads API's extend Audio & Models API's Mar 7, 2025
@zygiert1990 zygiert1990 marked this pull request as ready for review March 7, 2025 09:47
@zygiert1990 zygiert1990 requested a review from adamw March 7, 2025 09:47
@zygiert1990 zygiert1990 force-pushed the extend_audio_models_threads_apis branch from e516d8f to 82d87e4 Compare March 10, 2025 09:20
@adamw adamw merged commit 11a69ac into master Mar 10, 2025
9 checks passed
@adamw adamw deleted the extend_audio_models_threads_apis branch March 10, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants