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.

Sign up for GitHub

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

Jump to bottom

Feature: Add openai_api_model attribute to Doctran models #7868

Merged
merged 4 commits into from
Jul 20, 2023
Merged

Feature: Add openai_api_model attribute to Doctran models #7868

merged 4 commits into from
Jul 20, 2023
+28 −5

Conversation

aidendle94
Copy link
Contributor

@aidendle94 aidendle94 commented Jul 18, 2023 •
edited
Loading

  • Description: Added the ability to define the open AI model.
  • Issue: Currently the Doctran instance uses gpt-4 by default, this does not work if the user has no access to gpt -4.
  • rlancemartin, @eyurtsev, @baskaryan

@vercel
Copy link

vercel bot commented Jul 18, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2023 2:01pm

@dosubot dosubot bot added the 🤖:improvement Medium size change to existing code to handle new use-cases label Jul 18, 2023
@vercel vercel bot temporarily deployed to Preview July 18, 2023 05:24 Inactive
@vercel vercel bot temporarily deployed to Preview July 19, 2023 00:50 Inactive
@vercel vercel bot temporarily deployed to Preview July 19, 2023 04:24 Inactive
@baskaryan baskaryan added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Jul 19, 2023
@vercel vercel bot temporarily deployed to Preview July 20, 2023 14:01 Inactive
@baskaryan baskaryan merged commit 1d7414a into langchain-ai:master Jul 20, 2023
@yousenwang
Copy link

yousenwang commented Aug 21, 2023 •
edited
Loading

Is it possible to have LLMs other than OpenAI to use this feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baskaryan baskaryan

Assignees
No one assigned
Labels
🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aidendle94 @yousenwang @baskaryan