-
Notifications
You must be signed in to change notification settings - Fork 815
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
[Draft]Add Multimodal RAG notebook #2497
base: latest
Are you sure you want to change the base?
[Draft]Add Multimodal RAG notebook #2497
Conversation
openvino-dev-samples
commented
Nov 1, 2024
•
edited
Loading
edited
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
5f24b3f
to
e09b544
Compare
2050dc7
to
f72dab2
Compare
transfer to optimum-intel transfer to optimum-intel
f72dab2
to
669ad71
Compare
@openvino-dev-samples for me everything looks good, thanks. Couple of comments: Is there any plans to integrate OV Visual Language models directly in llama-index? |
Thanks for your review, the integration is already done in llama-index https://docs.llamaindex.ai/en/stable/examples/multi_modal/openvino_multimodal/ BTW is there an example for phi3-vision's accuracy aware quantization ? |
I'll add that an algorithm itself needs to be specified, e.g. Also, the default number of samples of 128 might be too large, so it can be reduced with |
Hi, as my test, the accuracy with this configuration is not satisfied: |
add load image function
ccede65
to
655ab9c
Compare
Thanks for the information! Have you compared it against the configuration below?
Yes, this configuration brings more reasonable responses compared to optimum-cli |
update the method of audio extraction
cce7f97
to
7dffa6a
Compare
update the screenshot display method
2a20586
to
7691683
Compare
hi @eaidova looks the CI is out of resource to validate this notebook, any suggestions ? thanks |
@openvino-dev-samples I can suggest to try internvl2-1b-instruct or nano-llava for ci testing, it is small enough to fit into precommit |
fix ci issues
9c30099
to
7207e0b
Compare
Sorry, I made a mistake before. The result looks good with this accuracy aware config now |
update with accruaracy aware quantization
c4dcb16
to
89f9ec5
Compare
solve conflict
2e02d4f
to
a868ab5
Compare