-
Notifications
You must be signed in to change notification settings - Fork 164
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
Feature/add app field to config #175
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
d845591
to
9836843
Compare
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.
emrgnt-cmplxty
added a commit
that referenced
this pull request
Mar 15, 2024
* feat: introduce Config class and update E2EPipelineFactory to use it (#172) * add changes * decouple config * Feature/blast web and chat (#173) * add changes * decouple config * blast web and chat * Feature/prompt provider and type reno (#174) * modify * mostly complete. * tweak types * Feature/add app field to config (#175) * modify * mostly complete. * tweak types * fix config name issue, add app field * tweak imports and all that * Feature/add redis config option (#176) * modify * mostly complete. * tweak types * add redis config option * erase cruft * Feature/add redis config option (#177) * modify * mostly complete. * tweak types * add redis config option * erase cruft * update package deps (#178)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
This PR introduces a new
PromptProvider
abstraction for generating prompts, a newR2RConfig
class for configuration management, updates theLLMProvider
interface for streaming completions, and enhances theLoggingDatabaseConnection
for fetching a limited number of logs.Key points:
PromptProvider
abstraction and used it inRAGPipeline
and its subclasses.R2RConfig
class for loading and validating the configuration.E2EPipelineFactory
to useR2RConfig
.LLMProvider
interface and its implementations (OpenAILLM
andLiteLLM
) to support streaming of completions.LoggingDatabaseConnection
to support fetching a limited number of logs.Generated with ❤️ by ellipsis.dev