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
Current limitations for auto-generated pipelines for component #286
Comments
I think 3 and 4 should be easier to fix. 1 and 2 is nice to have but could be difficult to cover all the edge cases. |
/cc @ckadner |
ckadner
added a commit
to ckadner/mlx
that referenced
this issue
Jan 19, 2022
* Many kubeflow/pipelines components do not have a description * Replace characters in component parameter names that are not allowed in Python identifiers Related machine-learning-exchange#286 Signed-off-by: Christian Kadner <[email protected]>
mlx-bot
pushed a commit
that referenced
this issue
Jan 20, 2022
* Many kubeflow/pipelines components do not have a description * Replace characters in component parameter names that are not allowed in Python identifiers Related #286 Signed-off-by: Christian Kadner <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
Here is the list of limitations for the current code generation:
"
into single quote'
for all the parameter values. This doesn't work for some components when users need to pass in JSON string with double quote around their keys and values. (e.g. https://github.com/kubeflow/pipelines/blob/master/components/contrib/json/Combine_lists/component.yaml)Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: