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

draft: Added Model Options #99

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

Arindam200
Copy link
Contributor

Description

In this PR I have fetched the available LLM models and added a Dropdown to switch between different LLM Models.

To Do:

  • Integrate the selected LLM model with Copilot

ScreenShot:

image

This PR fixes #89

@Arindam200 Arindam200 marked this pull request as draft February 26, 2024 18:00
}: {
query: string;
relevant: string;
// model: string;
Copy link
Contributor

Choose a reason for hiding this comment

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

this is correct here to define this for the askQuestion() - then you can pass in the model ID where you have the model: selectedModel. You will just have to only use the cloud models here (for now) since we are not handling the download and usage of any local model.

Using the local models requires the modelProgressDownloadController file that you can see an example of here to monitor their download status so they can actually be passed into your question logic.

For now i would just stick to the cloud models:

  • OpenAI models (4 models)
  • Palm2 models (2 models)
  • gemini models (1 model)

Copy link
Contributor

Choose a reason for hiding this comment

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

@shivaypiece shivaypiece self-requested a review February 27, 2024 00:41
@mason-at-pieces
Copy link
Collaborator

Hey @Arindam200 Could I get on update on this pr as well? Don't want them sitting here for too long

@Arindam200
Copy link
Contributor Author

Sorry for the Delay!

Will update the changes shortly

@mason-at-pieces
Copy link
Collaborator

Thank you! Just want to ensure that this project stays active 👍

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.

Add capability for Switching between LLM Models
4 participants