alexrudall / ruby-openai

OpenAI API + Ruby! 🤖❤️ NEW: Assistant Vector Stores
MIT License
2.73k stars 321 forks source link

Support for Assistants, Threads, Messages and Runs #361

Closed Haegin closed 10 months ago

Haegin commented 10 months ago

All Submissions:

~I've tried to keep to project conventions, though did add pry-byebug as an upgrade to byebug for easier debugging while I was working on the project. Happy to remove that if you'd prefer.~ pry-byebug doesn't support Ruby 2.6 and while it's no longer supported, I don't want to force a major version bump for this change so I've removed it.

alexrudall commented 10 months ago

@Haegin thanks for your work here! If you address the above comments I'll look to release this ASAP

jbeker commented 10 months ago

I'm not sure if this should be addressed in this PR or another, but the code in file uploads that tries to validate the uploaded file as JSON prevents the ability to upload arbitrary files for use with Assistants.

codenamev commented 10 months ago

I'm not sure if this should be addressed in this PR or another, but the code in file uploads that tries to validate the uploaded file as JSON prevents the ability to upload arbitrary files for use with Assistants.

I've added Assistant File support in #356

jcarrpries commented 10 months ago

Can we get a final approval and merge on this @alexrudall Looking to add this support into my project asap.

swistaczek commented 10 months ago

@alexrudall I also reviewed it on my end, and I think this looks great. Let us know when you think we can access this one upstream. Thank you!

alexrudall commented 10 months ago

Big thanks all - released in v6.1

dandailey commented 10 months ago

Excited to use this! Any docs/info on how?

alexrudall commented 10 months ago

@dandailey PRs welcome on adding these endpoints to the README :D https://github.com/alexrudall/ruby-openai/issues/376

alexrudall commented 10 months ago

@dandailey basically you make an assistant like this and then run it like this