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

Add a "cache" parameter to the pipeline to easily toggle this feature in KFP between runs #136

Open
MichaelClifford opened this issue Oct 30, 2024 · 2 comments
Assignees
Labels

Comments

@MichaelClifford
Copy link
Collaborator

MichaelClifford commented Oct 30, 2024

There are instance while debugging where you want to ensure caching is being used for steps that we typically don't want caching in order to speed up testing of steps that occur lower in the pipeline. Instead of hard coding the boolean value like this, set_caching_options(False), we should add it as a variable that gets exposed as a pipeline parameter so that it can be toggled on or off by the user on the DSP dashboard.

@MichaelClifford MichaelClifford self-assigned this Oct 30, 2024
@MichaelClifford MichaelClifford changed the title Add a "cache" parameter to the pipeline to easily toggle this feature in KFP between runs Feature: Add a "cache" parameter to the pipeline to easily toggle this feature in KFP between runs Oct 30, 2024
@MichaelClifford MichaelClifford changed the title Feature: Add a "cache" parameter to the pipeline to easily toggle this feature in KFP between runs Add a "cache" parameter to the pipeline to easily toggle this feature in KFP between runs Oct 30, 2024
@MichaelClifford
Copy link
Collaborator Author

Looks like this is actually currently not possible. Any function that impacts how the kube resource is defined needs to be set at IR compile time and cannot be used as a pipeline parameter. This is an issue KFP is aware of and there should be a fix sometime in the future.

@MichaelClifford
Copy link
Collaborator Author

As in intermediate measure, I will add a single variable that will be used for each .set_caching_option call. This will make toggling cache easier, but users will still have to compile and upload a new pipeline.yaml between each change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant