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

LlamaIndex and Gemini Quickstart #7

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lucifertrj
Copy link

This PR contains a notebook: where anyone can get started with LlamaIndex with Gemini as the LLM.

  • Approach-1: Using Image URL as input
  • Approach-2: Using local image

@MarkDaoust
Copy link
Contributor

Manually confirmed CLA. All good there.

@lucifertrj
Copy link
Author

Manually confirmed CLA. All good there.

Hi @MarkDaoust

Thank you for the confirmation. The review hasn't been reviewed to merge

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@TYMichaelChen
Copy link
Collaborator

Am I doing something wrong here? I get an error: ModuleNotFoundError: No module named 'llama_index.multi_modal_llms.generic_utils'
image

Copy link
Contributor

@MarkDaoust MarkDaoust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need images that are clearly open-licensed. We can't merge it with these images.

@lucifertrj
Copy link
Author

We need images that are clearly open-licensed. We can't merge it with these images.

Sure, understood, will modify it

@lucifertrj
Copy link
Author

lucifertrj commented Apr 11, 2024

Am I doing something wrong here? I get an error: ModuleNotFoundError: No module named 'llama_index.multi_modal_llms.generic_utils' image

LlamaIndex version is changed, we need to install this:

!pip install llama-index-multi-modal-llms-gemini
!pip install llama-index-llms-gemini

I will update the notebook, with free open licensed images and updated modules to run the Quickstart guide

@lucifertrj
Copy link
Author

@TYMichaelChen

Few changes to note:

  • SimpleDirectoryReader is now imported via llama_index.core
  • from llama_index.multi_modal_llms.generic_utils import load_image_urls is now called via llama_index.core i.e., from llama_index.core.multi_modal_llms.generic_utils import load_image_urls

@MarkDaoust

Used Open Licenses images from Unsplash

Copy link
Contributor

@MarkDaoust MarkDaoust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please delete the ".DS_Store" file.

For external integrations we're going to start an /examples/integrations directory. Can you create the directory and move this from /quickstarts/ to examples/integrations?

Thanks.

@lucifertrj
Copy link
Author

Please delete the ".DS_Store" file.

For external integrations we're going to start an /examples/integrations directory. Can you create the directory and move this from /quickstarts/ to examples/integrations?

Thanks.

Done

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.

None yet

4 participants