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

Integrate Gemini 1.0 Pro Chat Model and Google Embedding -001 Model for Enhanced Perplexica Capabilities #284

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Nipurn123
Copy link

This pull request that I have created introduces the integration of the Gemini 1.0 Pro Chat Model and the Google Embedding -001 Model into our Perplexica application . The following changes have been made to enhance our capabilities in generative AI and embeddings.

Key Changes

  • New Models Added:

    • Integrated Gemini 1.0 Pro Chat Model for enhanced conversational capabilities.
    • Integrated Google Embedding -001 Model for improved text embeddings.
  • File Modifications:

    • src/config.ts:
      • Added configuration settings for the Gemini API key.
    • src/lib/providers/gemini.ts:
      • Implemented functions to load the Gemini chat and embedding models.
      • Error handling added for model loading failures.
    • src/lib/providers/index.ts:
      • Updated the chat model and embedding model providers to include Gemini.
    • src/routes/config.ts:
      • Updated routes to handle Gemini API key retrieval.
    • ui/components/SettingsDialog.tsx:
      • Added input field for users to enter their Gemini API key in the settings dialog.

@Nipurn123
Copy link
Author

Nipurn123 commented Jul 24, 2024

Hi, I realised that I forgot to include the dependency @langchain/google-genai, so I've created this new pull request(#285) to address that. Additionally, since this is my first genuine PR, I would greatly appreciate it if you could take the time to review all the commits thoroughly. Thank you!

@Nipurn123
Copy link
Author

I've made this change to the package.json within the same PR. I apologize for the cumbersome process I chose earlier to update in PR #285.

@ItzCrazyKns
Copy link
Owner

Hi, the changes LGTM. But you've added the next dependency in the package file of the backend.

@Nipurn123
Copy link
Author

Hey I have removed that next dependency which by the way I am not sure why I have added before , but I have removed it from the backend package.json file

@ItzCrazyKns
Copy link
Owner

Hi, thanks. I will merge it after the testing

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.

2 participants