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

Config: move createSummaryOfFiles model selection to config #8

Closed
hatkyinc2 opened this issue Apr 6, 2023 · 2 comments
Closed

Config: move createSummaryOfFiles model selection to config #8

hatkyinc2 opened this issue Apr 6, 2023 · 2 comments

Comments

@hatkyinc2
Copy link
Collaborator

No description provided.

@fjrdomingues
Copy link
Owner

Hi, @hatkyinc2 what do you mean by "module" in this context?

@hatkyinc2 hatkyinc2 changed the title Config: move createSummaryOfFiles module selection to config Config: move createSummaryOfFiles model selection to config Apr 6, 2023
@hatkyinc2
Copy link
Collaborator Author

Sorry should have been model

Already done in that linked merged PR, closing.

The generic idea here and in #9 is to remove hardcoded stuff and allow for more flexibility, some of the reasons:

  • Not everyone has access to all the models (gpt4)
  • Not all models cost the same.
  • Not all models produce the same results and people may want to try others.
  • openAI will keep changing models. (gpt5 is in training for a while already rumors say)
  • Going forward I'm sure flexibility to replace OpenAI with other modules would be used by people even if we are not implementing it yet, it's a good path. Personally, I'd want to try gpt4all at some point. A sample list from langchain https://python.langchain.com/en/latest/modules/models/llms/integrations.html

fjrdomingues pushed a commit that referenced this issue Apr 7, 2023
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

No branches or pull requests

2 participants