-
Notifications
You must be signed in to change notification settings - Fork 1
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 LangChain with FastAPI #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, good job! Just added a few very small remarks.
Tested it successfully with OpenAI (Azure got some permission issues with my account, have to get that fixed first).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should be omittable since we are already ignoring .env
-files with the .gitignore
of the base folder.
@@ -0,0 +1,62 @@ | |||
# Intelligence Service | |||
|
|||
This is a FastAPI service for interfacing with LangChain and other machine learning services. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would love to have an additional more concrete description of which services we are currently using (e.g. OpenAI/Azure OpenAI, GPT-3.5-turbo, etc.).
load_dotenv() | ||
|
||
class Settings: | ||
APP_NAME: str = "Intelligence Service" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason you explicit typing it as str
?
@milesha you can stack PRs by branching of the |
Description
This PR is based on the PR for the FastAPI setup
Summary
Motivation
To effectively interface with large language models (LLMs), we need to integrate LangChain into our FastAPI intelligence service
Related Issue
Integrate LangChain with FastAPI Issue #32
Checklist
General
Client (if applicable)
Server (if applicable)
Screenshots (if applicable)
Attach screenshots here.