kedro-org / kedro-viz

Visualise your Kedro data and machine-learning pipelines and track your experiments.
https://demo.kedro.org
Apache License 2.0
646 stars 106 forks source link

Fix broken URL when active pipeline name changes on initial load #1914

Closed ravi-kumar-pilla closed 1 month ago

ravi-kumar-pilla commented 1 month ago

Description

Resolves #1879

pid_fix

Development notes

QA notes

Checklist

ravi-kumar-pilla commented 1 month ago

Hi @rashidakanchwala ,

Could you please remove the CircleCI integration for this repo ? Thank you

image

jitu5 commented 1 month ago

@ravi-kumar-pilla to remove CircleCI Pipeline check you need to sync with main as you branch will have .circleCI config

ravi-kumar-pilla commented 1 month ago

@ravi-kumar-pilla to remove CircleCI Pipeline check you need to sync with main as you branch will have .circleCI config

I already did - https://github.com/kedro-org/kedro-viz/commit/c26b16e7b95ebf82d73e1e0d2de7fc8d76c2dcea

jitu5 commented 1 month ago

@ravi-kumar-pilla to remove CircleCI Pipeline check you need to sync with main as you branch will have .circleCI config

I already did - c26b16e

@ravi-kumar-pilla Rashida recently did it and after that I sync with main and that check is gone. https://github.com/kedro-org/kedro-viz/pull/1894 Please push something to this branch.

rashidakanchwala commented 1 month ago

i enabled it now, maybe you need to do another push to re-run the workflows.