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

feat: fine-grain resource config at skeleton level #13

Merged
merged 13 commits into from
Feb 8, 2024
Merged

feat: fine-grain resource config at skeleton level #13

merged 13 commits into from
Feb 8, 2024

Conversation

justinthelaw
Copy link
Contributor

  • Add README details
  • Moved GPU, CPU, and Memoray resource control to skeleton
  • GPU enablement and resources always prompted

@justinthelaw justinthelaw added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 2, 2024
@justinthelaw justinthelaw linked an issue Feb 2, 2024 that may be closed by this pull request
GPU configuration for all model backends #12
Closed
@justinthelaw justinthelaw changed the title feat: fine-grain resource configuration feat: fine-grain resource config at skeleton level Feb 2, 2024
@justinthelaw justinthelaw marked this pull request as ready for review February 2, 2024 21:58
@YrrepNoj YrrepNoj mentioned this pull request Feb 6, 2024
Merged
@justinthelaw justinthelaw requested a review from YrrepNoj February 6, 2024 14:59
@justinthelaw justinthelaw requested a review from a team as a code owner February 7, 2024 18:29
YrrepNoj
YrrepNoj reviewed Feb 7, 2024
View reviewed changes
chart/values.yaml Outdated Show resolved Hide resolved
@justinthelaw justinthelaw requested a review from YrrepNoj February 7, 2024 20:08
YrrepNoj
YrrepNoj reviewed Feb 7, 2024
View reviewed changes
zarf.yaml Outdated Show resolved Hide resolved
YrrepNoj
YrrepNoj requested changes Feb 8, 2024
View reviewed changes
zarf.yaml Outdated Show resolved Hide resolved
zarf.yaml Outdated Show resolved Hide resolved
@YrrepNoj YrrepNoj mentioned this pull request Feb 8, 2024
Closed
@justinthelaw justinthelaw requested a review from YrrepNoj February 8, 2024 15:10
@justinthelaw justinthelaw merged commit a01c9ae into main Feb 8, 2024
1 check passed
@justinthelaw justinthelaw deleted the 12-gpu-configuration-for-all-model-backends branch February 8, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YrrepNoj YrrepNoj YrrepNoj approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GPU configuration for all model backends
2 participants
@justinthelaw @YrrepNoj