Revital Sur

Results 10 comments of Revital Sur
trafficstars

Here is a example of an error we get trying to pass input parameters as argument to kfp component: ``` import kfp.dsl as dsl from kfp import compiler @dsl.component(base_image="python:3.10") def...

> @revit13 Can you please rebase and resolve the conflicts? @chensun Done, Thanks

> @revit13 , we'll need a document that explains it @roytman I added description to kfp/doc/multi_transform_pipeline.md.

Hi @shivdeep-singh-ibm can you please also update .pre-commit-config.yaml file? (remove `IBM Confidential` header)

Please consider applying pre-commit only on the changes files in the pull request using ` --from-ref` and `--to-ref`. This way the fixes can be applied incrementally.

> Lets follow what we have here https://github.com/IBM/data-prep-kit/tree/dev/examples for order. All the existing test datasets we have can be used. Let me know if you need me to point you...

It seems that the following approach could help: 1) Create a shared venv in the main repo where the data-prep-lab libraries are installed. This shared venv will be re-created only...

@shivdeep-singh-ibm it seems that the `workflow-` rules in transforms/code/license_check/Makefile needs to be changed (they refer to `kfp_ray/v1` directory which does not exist). This causes the CI fail: Please see the...

I encounter this error on Ubuntu 20.04.6 LTS when deploying standalone kfp 1.8.5 on a Kind cluster. Any help in resolving it is appreciated ``` [pod/metadata-grpc-deployment-b45564d7d-rqf46/container] WARNING: Logging before InitGoogleLogging()...

LGTM. btw you also need to update transforms/pyproject.toml