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

Update inputs of KFP example components to use KFP base types #115

Open
kiersten-stokes opened this issue Nov 30, 2022 · 0 comments
Open

Comments

@kiersten-stokes
Copy link
Member

In KFP v1, any value can be given for the type field of an input. This can be problematic during pipeline compile if parameters are passed to a pipeline. An InconsistentTypeException will be thrown if the type hint of a parameter does not match the type of the input defined in the component YAML.

For example, in the inputs for the Download Data example component, the values of string and URI are given as types for the inputs. While it is clear that these types should be strings, an InconsistentTypeException is thrown during compile for a pipeline with a parameter type-hinted as str in the Python DSL because KFP defines its string type as String with a capital S.

We should update all our example component resources to only use KFP base types so that these components continue to work 'out of the box' when support for pipeline parameters is available in Elyra (elyra-ai/elyra#3001).

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

No branches or pull requests

1 participant