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

Remove unused values and change GenAIExamples default #397

Merged
merged 2 commits into from
Sep 5, 2024

Conversation

lianhao
Copy link
Collaborator

@lianhao lianhao commented Sep 5, 2024

Description

  • helm: remove unused values
  • change GenAIExamples to download models by default

Issues

n/a.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
Changes the default example for K8S manifests for GenAIExamples to
always download the model to avoid extra setup requirements for the end
users.

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
@lianhao lianhao requested review from daisy-ycguo and removed request for KfreeZ and zhlsunshine September 5, 2024 05:04
@lianhao lianhao merged commit 26f9b16 into opea-project:main Sep 5, 2024
14 checks passed
@lianhao lianhao deleted the remove_unused_env branch September 5, 2024 06:27
Yugar-1 pushed a commit to Yugar-1/GenAIInfra that referenced this pull request Sep 5, 2024
* helm: remove unused values

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>

* change GenAIExamples to download models by iteself

Changes the default example for K8S manifests for GenAIExamples to
always download the model to avoid extra setup requirements for the end
users.

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>

---------

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
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

Successfully merging this pull request may close these issues.

3 participants