Commit Graph

13 Commits

Author SHA1 Message Date
Connor McCarthy c7301c449c
feat(sdk)!: use yaml for IR package (#7431)
* change serialization format from json to yaml

* refactor: use guard clause for clarity

* convert test data from json to yaml

* add and update tests

* permit deprecated compile to json

* re-add json test data

* update compiler tests

* refactor Compiler._write_pipeline_spec_file

* update golden snapshots

* update sample output extensions

* test file extension exceptions and warnings

* update release notes

* remove json golden snapshots
2022-03-21 21:45:31 +00:00
Yaqi Ji 8bb0d0e025
feat(sdk)!: move v2 to main namespace (#7376)
* feat(sdk)!: move v2 to main namespace

* edit samples
2022-03-04 21:56:51 +00:00
Chen Sun ed3453f794
chore: update comments in a pipeline sample (#6601)
* Update lightweight_python_functions_v2_pipeline.py

* Update lightweight_python_functions_v2_pipeline.py

* Update lightweight_python_functions_v2_pipeline.py

* Update mock_lightweight_python_functions_v2_pipeline.json

* Update lightweight_python_functions_v2_pipeline.json

* Update mock_lightweight_python_functions_v2_pipeline.json
2021-09-23 11:29:36 -07:00
Ajay Gopinathan f3f383c2ff
chore(sdk): Refactor and move all v2 related code to under the v2 namespace. (#6358)
* Refactor and move all v2 related code to under the v2 namespace.

Most of the changes are around imports and restructuring of the
codebase. While it looks like a lot of code was added, most of the code
already existed and was simply moved or copied over to v2. The only
exceptions are:
- under kfp/v2/components/component_factory.py: some helper functions
  were copied with simplification from _python_op.py
- we no longer strip the `_path` suffix in v2 components.

Note: there is still some duplication of code (particularly between
component_factory.py and _python_op.py), but it's ok for now since we
intend to replace some of this with v2 ComponentSpec + BaseComponent.

* Update setup.py.

* update tests.

* revert accidental change of gcpc

* Fix component entrypoint.

* Update goldens.

* fix tests.

* fix merge conflict.

* revert gcpc change.

* fix tests.

* fix tests.

* Add type aliases for moved files.

* merge and update goldens.
2021-08-17 19:25:37 -07:00
Yuan (Bob) Gong afd28c605d
test(v2): add more samples. Part of #6150 (#6353)
* test(v2): add all samples using parameter + artifact passing features to v2 engine

* fix some samples

* fix output file

* comment failing samples

* improve comments
2021-08-16 21:38:41 -07:00
Ajay Gopinathan c5591f2652
feat(sdk): Enable the use of Executor from KFP package for v2 lightweight components (#6202)
* Enable the use of v2 Executor from KFP package.

This avoids the need to paste Executor code for v2 components into
the entrypoint of lightweight components.

* Update tests.

* quote package path.

* quote package path.

* escape fragment

* Update golden files.

* update goldens.

* Cleanup.

* More cleanup.

* update goldens.

* style updates.

* clean up sample tests.

* fix trailing comma

* Try using signed urls.

* fix

* fix presubmit

* try /bin/bash for Makefile shell

* undo signed url change.

Co-authored-by: Yuan (Bob) Gong <4957653+Bobgy@users.noreply.github.com>
2021-08-05 14:53:03 -07:00
Yuan (Bob) Gong f3b369e153
fix: v2compat - handle parameter values with special characters stably. Fixes #5830 (#5973)
* fix(launcher): handle parameter values with special characters stably

* include new test case

* add tensorboard minio test case

* fix go unit tests

* update test golden

* address feedback

* fix tests
2021-07-08 22:02:23 -07:00
Chen Sun 3f45f78a3d
fix(sdk): Fix default value of component input not picked up. Fixes #5880. (#5903)
* Fix default value of component input not picked up

* fix v2 compat
2021-06-23 13:25:18 -07:00
Yuan (Bob) Gong 834a5cc80f
test(sdk/v2compat): reproduce v2 py component default value does not work (#5879)
* test(sdk/v2compat): reproduce v2 py component default value does not work

* fix

* fix
2021-06-18 08:43:15 -07:00
Yuan (Bob) Gong 77c1de8152
fix(launcher): duplicate input artifacts logged (#5836)
* fix(launcher: input artifact logging when more than one

* fix

* fix

* disable presubmit v2 go test
2021-06-11 21:39:52 +08:00
Alexey Volkov cc83e1089b
Assigned copyright to the project authors (#5587) 2021-05-05 13:53:22 +08:00
Ajay Gopinathan b46f1b86f1
fix(sdk): Update syntax for lightweight components v2. (#5568)
* Update syntax for lightweight components v2.

Use Input[T] and Output[T] for representing input/output artifacts
of type T. This enables users to see these annotations as merely the
classes T, enabling autocompletion, reducing verbosity and simplifying
I/O types code. Users also no longer

Other changes include:
- no longer need to use .get() to obtain the artifact class
- no auto-creation of the `data` file for output artifacts. This enables
  artifacts to be either directories or files.

* fix for Py 3.6

* Remove stray file.

* Add error message if Input/Output are used with non Artifact types.

* Fix e2e test.
2021-04-29 19:27:30 -07:00
Yuan (Bob) Gong 02dbcfa062
test: more samples in v2 sample test (#5431)
* test: set up sample test for many samples

* test: rm loop_* tests from v1 sample test, they are already covered in v2 sample test

* fix condition pipeline for basic e2e test

* remove condition from e2e test
2021-04-09 08:34:03 -07:00