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

feat(sdk): Use google.protobuf.Value in v2 for passing parameters. #6804

Merged
merged 26 commits into from
Oct 28, 2021
Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
4966b0b
Use google.protobuf.Value in v2 for passing parameters.
neuromage Oct 26, 2021
8f14fb1
retest samples.
neuromage Oct 26, 2021
728f922
Fix tests.
neuromage Oct 27, 2021
5b13589
Merge branch 'master' into protobuf-value.
neuromage Oct 27, 2021
cf3407d
Update release, more cleanup.
neuromage Oct 27, 2021
7e6d18f
Use github.com/kubeflow/pipelines/api from same repo.
neuromage Oct 27, 2021
bde385f
Run go mod tidy
neuromage Oct 27, 2021
ac5f6e3
chore: go mod tidy
Bobgy Oct 27, 2021
fd8ee77
fix v2 compile error and clean up unused code
Bobgy Oct 27, 2021
216944a
pr comments.
neuromage Oct 27, 2021
87319b8
Merge branch 'protobuf-value' of github.com:neuromage/pipelines into …
neuromage Oct 27, 2021
107a492
Merge branch 'master' into protobuf-value
neuromage Oct 27, 2021
df770b8
Merge branch 'master' into protobuf-value
neuromage Oct 27, 2021
9b5e012
update goldens
neuromage Oct 27, 2021
c8d49a2
Fix metadata recording.
neuromage Oct 27, 2021
de1f943
Update kfp mlmd client.
neuromage Oct 27, 2021
bfc380b
fix test again
neuromage Oct 27, 2021
ad3e7f9
another try.
neuromage Oct 27, 2021
917d979
chore: migrate v2 DAG driver input parameters to protobuf.Value + sma…
Bobgy Oct 28, 2021
b0718ec
fix v2 launcher + clean up
Bobgy Oct 28, 2021
1df554a
fix a compile error
Bobgy Oct 28, 2021
24360f2
fix a few more tests
Bobgy Oct 28, 2021
7b9a681
fix number parsing
Bobgy Oct 28, 2021
547747e
clean up
Bobgy Oct 28, 2021
f2426b0
disable cache_v2 test.
neuromage Oct 28, 2021
35f8448
Merge branch 'protobuf-value' of github.com:neuromage/pipelines into …
neuromage Oct 28, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix a compile error
Bobgy committed Oct 28, 2021

Verified

This commit was signed with the committer’s verified signature.
RaphiMC RK_01
commit 1df554ac869fe8a2ef523f43dfd0868b1f712373
2 changes: 1 addition & 1 deletion v2/component/launcher.go
Original file line number Diff line number Diff line change
@@ -475,7 +475,7 @@ func (l *Launcher) dumpOutputParameters(executorOutput *pipelinespec.ExecutorOut
wrap := func(err error) error {
return fmt.Errorf("failed to dump output parameter %q in executor output to disk: %w", name, err)
}
value, err := pbValueToText(parameter)
value, err := metadata.PbValueToText(parameter)
if err != nil {
return wrap(err)
}