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

[Sample] add a sample for multiple outputs #1517

Closed
hongye-sun opened this issue Jun 15, 2019 · 6 comments
Closed

[Sample] add a sample for multiple outputs #1517

hongye-sun opened this issue Jun 15, 2019 · 6 comments
Assignees

Comments

@hongye-sun
Copy link
Contributor

Add a basic sample to demonstrate returning multiple outputs from a single step. It seems that all samples today are have only single output.

@Ark-kun
Copy link
Contributor

Ark-kun commented Jun 17, 2019

The Lightweight components sample has multiple outputs https://github.com/kubeflow/pipelines/blob/master/samples/notebooks/Lightweight%20Python%20components%20-%20basics.ipynb

But I agree that we need more components and samples demonstrating that.

What do you think about just making a component like Split lines that can for example split table into train/test.

@Ark-kun
Copy link
Contributor

Ark-kun commented Jun 25, 2019

Can you look at the Lightweight sample? WDYT?

@zanedurante
Copy link
Contributor

zanedurante commented Jun 28, 2019

For me, the lightweight sample is alright -- I only found it by first finding this GitHub issue though.
Is there a way to do this using compiler.build_python_component() for non lightweight components? I am following the example but I am getting Output type not supported and supported types are [int, float, str, bool] (using compiler.build_python_component()). It works if I use func_to_container_op().

@zanedurante
Copy link
Contributor

This issue has been documented here: #1580

@jessiezcc
Copy link
Contributor

/area samples

@gaoning777
Copy link
Contributor

I'm including this as part of the sample efforts.

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

6 participants