-
-
Notifications
You must be signed in to change notification settings - Fork 193
Add video input file support #260
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
base: main
Are you sure you want to change the base?
Conversation
I've only tested this with Gemini 1.5 Flash and it works. Updating the models and running the tests makes a lot of errors fail. What's the best way to proceed from here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @arnodirlam I'd love to add this to RubyLLM. Do you have the means to test things out with API keys? There's one little thing to change too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should rather be in chat_content_spec.rb
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 228ab17.
Oh and check off the items that you did in the quality check section of the PR. All are required. |
c1bbf45
to
9604320
Compare
dcd8183
to
f8c4655
Compare
…; Update capabilities
I've addressed the remaining points now.
EDIT: Just realized |
Added a |
What this does
Enables RubyLLM to recognize and handle video files as attachments, in addition to existing support for images, audio, PDFs, and text.
Why?
Type of change
Scope check
Quality check
overcommit --install
and all hooks passmodels.json
,aliases.json
)API changes
Related issues
Fixes #259