We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
While I'm trying to lauch a KFP following the doc, I found that current URL concat way is not compatible when user set endpoint with a / in the end.
/
from kfp import dsl from kfp import client @dsl.component def addition_component(num1: int, num2: int) -> int: return num1 + num2 @dsl.pipeline(name='addition-pipeline') def my_pipeline(a: int, b: int, c: int = 10): add_task_1 = addition_component(num1=a, num2=b) add_task_2 = addition_component(num1=add_task_1.output, num2=c) endpoint = 'http://localhost:8080/' # <= endpoint with a / in the end kfp_client = client.Client(host=endpoint) run = kfp_client.create_run_from_pipeline_func( my_pipeline, arguments={ 'a': 1, 'b': 2 } ) url = f'{endpoint}/#/runs/details/{run.run_id}' print(url)
There will be a redundant '/' following 8080. http://localhost:8080//#/runs/details/your_run_id_here
8080
http://localhost:8080//#/runs/details/your_run_id_here
I think we can use urllib other than just a simple string concat to avoiding this possible problem.
urllib
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
While I'm trying to lauch a KFP following the doc, I found that current URL concat way is not compatible when user set endpoint with a
/
in the end.Current Doc Content
A Possible Mistake
Output
There will be a redundant '/' following
8080
.http://localhost:8080//#/runs/details/your_run_id_here
What We Can Do
I think we can use
urllib
other than just a simple string concat to avoiding this possible problem.The text was updated successfully, but these errors were encountered: