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

Large Multimodal Models #515

Closed
wants to merge 46 commits into from
Closed

Large Multimodal Models #515

wants to merge 46 commits into from

Conversation

BeibinLi
Copy link
Collaborator

@BeibinLi BeibinLi commented Nov 1, 2023

Why are these changes needed?

Large Multimodal Models. Right now, LLaVA is supported from agentchat.

Related issue number

Checks

@codecov-commenter
Copy link

Codecov Report

Merging #515 (a6cdcfa) into main (957ec00) will decrease coverage by 24.97%.
The diff coverage is 18.00%.

@@             Coverage Diff             @@
##             main     #515       +/-   ##
===========================================
- Coverage   41.75%   16.79%   -24.97%     
===========================================
  Files          21       23        +2     
  Lines        2632     2823      +191     
  Branches      592      620       +28     
===========================================
- Hits         1099      474      -625     
- Misses       1442     2347      +905     
+ Partials       91        2       -89     
Flag Coverage Δ
unittests 16.79% <18.00%> (-24.97%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
autogen/agentchat/__init__.py 100.00% <100.00%> (ø)
autogen/agentchat/multimodal_conversable_agent.py 30.90% <30.90%> (ø)
autogen/img_utils.py 17.46% <17.46%> (ø)
autogen/agentchat/contrib/llava_agent.py 6.32% <6.32%> (ø)

... and 13 files with indirect coverage changes

@BeibinLi BeibinLi closed this Nov 1, 2023
jackgerrits pushed a commit that referenced this pull request Oct 2, 2024
merging this in because its better than what we have. will make new pre for subsequent changes.
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.

3 participants