Commit Graph

9 Commits

Author SHA1 Message Date
Connor McCarthy 13736b85b6
chore(sdk): clean up kfp code style (#8027)
* remove unused imports

* use google as isort profile

* sort imports

* format with yapf

* clean end of file new line, trailing whitespace, double quoted strings
2022-07-15 00:06:34 +00:00
Connor McCarthy 4621aec146
chore(sdk): apply yapf formatting (#7414) 2022-03-16 20:34:36 +00:00
Yaqi Ji b693909a08
feat(sdk)!: move v1 to deprecated folder (#7291)
* chore(sdk): move v1 to deprecated

* fix testsg

* fix testsg

* fix setup.py

* fix test

* s'

* fix tests

* fix test

* retore v2 test changes

* fix py3.6 test

* fix py3.6 test

* fix py3.6 import fallback error

* remove deprecated

* fix samples test

* sample test

* fix samples

* add readme

* restroe test

* python require

* remove path

* fix tests

* inteegration tests

* remove tfx tests for dependency with kfp v1

* fix e2e

* fix e2e

* fix integration tests

* fix sampe

* move client down

* change to kfp

* add import alias

* fix

* runid

* fix dsl

* only use kfp for function

* revert train_until_good

* tfx test

* kfp

* try import

* onprem
2022-03-04 00:52:59 +00:00
Lin Yiming 36e627d28f
feat(sdk): Local runner supports additional docker options. Fixes #6598 (#6599)
* bump kfp-pipeline-spec to 0.1.11, tests for local runner will fail with kfp-pipeline-spec 0.1.9

* Local client supports additional docker options

* Format _local_client.py using yapf

* Add release note that local runner supports additional docker options
2021-09-24 07:58:37 -07:00
Chen Sun 2f19a26ffd
chore(sdk): Format all Python files under SDK folder. (#6501)
* Reformat sdk only using the new yapf config.

* Reformat docstrings using docformatter.

* update golden files to resolve diff caused by whitespaces

* fix some tests

* format .py files under sdk/python/tests using yapf

* additional docformatter

* fix some tests
2021-09-03 11:25:11 -07:00
Alexey Volkov cc83e1089b
Assigned copyright to the project authors (#5587) 2021-05-05 13:53:22 +08:00
Alexey Volkov d0cbca7629
fix(sdk): Local runner - Fixed the local runner on Windows (#5313)
Sume of the fixed issues could also affect Linux OSes.
2021-03-18 23:19:16 -07:00
Chen Sun 45c5c18716
fix(tests): Disable the SDK unittest that requires docker binary in local environment. (#5211) 2021-02-28 22:42:34 -08:00
Lin Yiming 8af15a3ad0
feat(sdk): Add local runner which will run ops in docker or locally. Fixes #1138 (#4983)
* add local runner which will run ops in docker or locally

* use str.format rather than f-string

* add some brief doc string in local client

* comment the unittest about running op in docker, which is not supported in CI env for now

* Add some brief docstring about DAG used in local client

* make graph/reverse_graph of DAG as property to keep them in sync

* make some methods of LocalClient static

* remove circular reference in local client

* Incapsulate artifact storage root in the constuctor of LocalClient

* Add Alpha notice for kfp.run_pipeline_func_locally

* Support list of local images in kfp.run_pipeline_func_locally

* make staticmethod to module level private method

* Trivial modification according to code review, some renaming or docstring

* local runner support components without '--' as argument prefix

* make output file of op in loop unique

* Local runner decides whether run component in docker or in local process base on ExecutionMode
2021-02-23 21:07:14 -08:00