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

chore(backend): use non deprecated accelerator type & count fields in driver #11375

Open
HumairAK opened this issue Nov 14, 2024 · 2 comments · May be fixed by #11409
Open

chore(backend): use non deprecated accelerator type & count fields in driver #11375

HumairAK opened this issue Nov 14, 2024 · 2 comments · May be fixed by #11409
Assignees
@mahdikhashan
Labels
area/backend good first issue
Milestone
KFP 2.4.0

Comments

@HumairAK
Copy link
Collaborator

Chore description

Driver should use resource_type and resource_count and move away from deprecated type and count fields. Api sec for these fields are here.

Driver calls for these fields are here.

This PR and comment for more context: #11373 (comment)

Labels

/area backend

Love this idea? Give it a 👍.
@HumairAK HumairAK added this to the KFP 2.4.0 milestone Nov 14, 2024
@tarilabs tarilabs mentioned this issue Nov 25, 2024
Open
@mahdikhashan
Copy link

mahdikhashan commented Nov 25, 2024 •
edited
Loading

I would like to take over this issue (as my first code pr) and start working on it. would it be possible to assign it to me? @HumairAK @tarilabs

@vikas-saxena02
Copy link

/assign @mahdikhashan

@mahdikhashan mahdikhashan linked a pull request Nov 26, 2024 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mahdikhashan mahdikhashan

Labels
area/backend good first issue
Projects
None yet
Milestone
KFP 2.4.0
Development

Successfully merging a pull request may close this issue.

chore(backend): use GetResourceType and GetResourceCount. Fixes #11375. mahdikhashan/pipelines
3 participants
@HumairAK @mahdikhashan @vikas-saxena02