Yusuke Nishioka
Yusuke Nishioka
@jd Hello. Excuse me for cutting in. What's the status of this PR? There's something I'd like to work on after this PR is merged.
If this fix is going to be included in 1.2, I'd say the release note is not necessary. We can just say "Added support for `grpc.aio`" as written in [1.2.0rc1...
@majorgreys @brettlangdon Sorry for this direct mention, I'm reaching out to you because you reviewed #3240. Would be great if you could review this PR when you're available as it...
@majorgreys @brettlangdon Hello, could you review this PR? it's been left behind for a long time.
@connor-mccarthy Thank you for your response. I tried with the following pipeline, pipeline ```py from kfp import dsl PIPELINE_NAME = "exit-handler-pipeline" LOCAL_PIPELINE_PATH = f"outputs/{PIPELINE_NAME}.yaml" @dsl.container_component def exit_op(status: str = dsl.PipelineTaskFinalStatus):...
@emmettbutler Thank you for your review. Some test suites failed but I didn't make any change other than grpc, I think it's OK to proceed.
@emmettbutler Could you run the test suites again?
@emmettbutler Thank you for running the test suites. I've updated the name of this PR, so could you run CI again?
Binaries for CUDA 10 is highly likely to be incompatible with CUDA 11 and we haven't provided binaries for CUDA 11 yet. Sorry for the inconvenience.
In our case, approved PRs are always excluded from the reminder even though we don't mark the option.