Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support getEmojis() with multiple input strings, switch to gpt-3.5-turbo #5

Merged
merged 1 commit into from
Mar 19, 2023

Conversation

bmish
Copy link
Owner

@bmish bmish commented Mar 18, 2023

Supporting bulk usage with multiple input strings could be more efficient and unlock future features like #6.

OpenAI recommends gpt-3.5-turbo as the best and cheapest model currently. Had to rewrite the request/response for that. Switched to this because our prompt became more complex.

Fixes #3.

@bmish bmish added the enhancement New feature or request label Mar 18, 2023
@bmish bmish force-pushed the text-array-input branch 2 times, most recently from f6387b2 to b70401c Compare March 19, 2023 00:26
@bmish bmish force-pushed the text-array-input branch from b70401c to 6857a3b Compare March 19, 2023 00:27
@bmish bmish marked this pull request as ready for review March 19, 2023 00:28
@bmish bmish changed the title Support getEmojis() for multiple strings at once, switch to gpt-3.5-turbo Support getEmojis() with multiple input strings, switch to gpt-3.5-turbo Mar 19, 2023
@bmish bmish merged commit c2051e1 into main Mar 19, 2023
@bmish bmish deleted the text-array-input branch March 19, 2023 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add bulk API to query for several results at once
1 participant