Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Create KFP artifacts using the SDK #333

Merged

Conversation

StefanoFioravanzo
Copy link
Member

This PR introduces a new SDK decorator that allows users to declare
the artifacts created during a step execution. The @artifact decorator
is thus supposed to be used on functions that have already been
decorated with the @step decorator.

Declaring step artifacts must be done before the step actually executes,
because Kale needs to create a pipeline that is aware of these outputs,
and properly update each step's output artifacts dictionary in the KFP
DSL.

Currently only HTML artifacts are supported.

Stefano Fioravanzo added 2 commits April 12, 2021 08:54
This commit introduces a new SDK decorator that allows users to declare
the artifacts created during a step execution. The `@artifact` decorator
is thus supposed to be used on functions that have already been
decorated with the `@step` decorator.

Declaring step artifacts must be done before the step actually executes,
because Kale needs to create a pipeline that is aware of these outputs,
and properly update each step's output artifacts dictionary in the KFP
DSL.

Currently only HTML artifacts are supported.

Signed-off-by: Stefano Fioravanzo <stefano@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>
Signed-off-by: Stefano Fioravanzo <stefano@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>
Stefano Fioravanzo added 2 commits April 12, 2021 10:19
Signed-off-by: Stefano Fioravanzo <stefano@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>
Signed-off-by: Stefano Fioravanzo <stefano@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>
@StefanoFioravanzo StefanoFioravanzo force-pushed the feature-stefano-up-sdk-kfp-artifacts branch from dcf6cce to 9ebd69f Compare April 12, 2021 08:19
@elikatsis elikatsis merged commit 394a5b5 into kubeflow-kale:master Apr 14, 2021
@elikatsis elikatsis deleted the feature-stefano-up-sdk-kfp-artifacts branch April 14, 2021 12:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants