Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

initial commit for official api usage #250

Closed
wants to merge 1 commit into from
Closed

Conversation

max298
Copy link
Collaborator

@max298 max298 commented Nov 13, 2023

I've spent some time checking out the official library from OpenAI. The assistants api (which is still in beta) looks like a perfect match for our use-case, as it uses threadIds which persist in order to keep the context.

Please also note that this merge would change the data saved inside the database (no more messageId), so I don't know if it's even possible to upgrade from a older version (and even if, you'd break your old threads).

This is not a "real" merge request - I just played around and got the bot in a working state (which is not well tested and also not well coded). I just thought it might be useful to have the code in case we ever decide to use the assistants api.

Initially I wanted to take a look at the gpt-4-vision-preview (see #247), however this model is not supported with the assistants api:

"The requested model 'gpt-4-vision-preview' cannot be used with the Assistants API."

I think it would be really cool to have the vision-feature within this bot, so I don't know how I feel about the assistants api.

@bertybuttface
Copy link
Collaborator

I really like this idea! We don't currently support vision anyway and OpenAI may yet add it to the Assistant API. Thanks for sharing your work so we can all start thinking about how this should work.

@max298 max298 closed this Nov 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants