-
Notifications
You must be signed in to change notification settings - Fork 136
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
Prediction Guard LVM component #676
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
Signed-off-by: sharanshirodkar7 <[email protected]>
sharanshirodkar7
requested review from
Spycsh and
lvliang-intel
as code owners
September 11, 2024 18:47
…pace, imports, and formatting Signed-off-by: sharanshirodkar7 <[email protected]>
@chensuyue PG LVM component ready to review. |
Signed-off-by: sharanshirodkar7 <[email protected]>
@chensuyue @lvliang-intel @Spycsh all checks passed. Ready for review. Thanks |
lvliang-intel
approved these changes
Sep 17, 2024
Spycsh
approved these changes
Sep 18, 2024
chensuyue
pushed a commit
that referenced
this pull request
Sep 18, 2024
* added files for PG lvm component Signed-off-by: sharanshirodkar7 <[email protected]> * Fix pre-commit issues: end-of-file, requirements.txt, trailing whitespace, imports, and formatting Signed-off-by: sharanshirodkar7 <[email protected]> * added package Signed-off-by: sharanshirodkar7 <[email protected]> --------- Signed-off-by: sharanshirodkar7 <[email protected]> (cherry picked from commit 1249c4f)
lkk12014402
pushed a commit
that referenced
this pull request
Sep 19, 2024
* Update set_env.sh to fix #670 * Update README.md fixed your_ip and host_ip * Update README.md fix your_ip and host_ip
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.
Description
This PR adds the LVMs component for Prediction Guard (PG) to the repository. The new component integrates various LVM capabilities to enhance the functionality of the Prediction Guard platform, allowing for more robust text generation and natural language processing features. The motivation behind this addition is to expand the library’s support for advanced language models and provide users with additional tools for their AI and ML workflows.
Issues
n/a
Type of change
New feature (non-breaking change which adds new functionality)
Dependencies
PREDICTIONGUARD_API_KEY needs to be added to CI/CD to test the microservice.
Tests
The following tests were run to verify the changes:
• Component functionality tests to ensure the new LVMs component works as expected.
• Integration tests to validate that the new component integrates seamlessly with the existing Prediction Guard ecosystem.
• Unit tests for each new function and module added as part of the LVMs component.