Hi all, long time I opened an issue in kedro-viz ...
# questions
i
Hi all, long time I opened an issue in kedro-viz since yesterday's release has broken our prod pipelines. I think the issue is because of the kedro-telemetry package being installed and it seems to have caused the order of our hooks' execution to get messed up. Reinstalling without kedro-viz and by extension kedro-telemetry solves the issue. https://github.com/kedro-org/kedro-viz/issues/2203
šŸ‘€ 2
h
Someone will reply to you shortly. In the meantime, this might help:
r
Thanks for bringing this up.
kedro-telemetry
is required dependency now. It was introduced in this - https://github.com/kedro-org/kedro-viz/pull/2112 Can i ask you a question, did you have
kedro-telemetry
installed before ? or it never was
i
Nope, it wasn't installed before. Do you have a view on how the presence of kedro-telemetry would interact with hook execution order?
It's an easy fix for us since we were being a bit lazy with our dependency groups, we're just removing kedro-viz from our prod deps, but I'm curious about what could've changed
r
i think if you can provide a traceback (whenever u can) it would help a lot because even I am confused at this point. but we will look into how kedro-telemetry is causing this.
i
I will share as soon as I can, but we don't have a traceback per se: Before we had a hook which runs an
after_context_created
hook which initializes a singleton containing certain info about the context which we use later. This hook now isn't running at the same point and the singleton is being accessed from within the pipeline registry before it has had time to initialize itself correctly.
r
thanks, I think the issue is more likely to be a kedro-telemetry issue. I will move your issue to kedro-telemetry repository for now and someone from the team will look into it.
kedro-telemetry
is now a required kedro dependency versions > 0.19.7 and on kedro-viz from versions > 10.1.0
i
Yeah that makes sense, thanks! I'll paste this extra context there later
šŸ‘ 1
We're on 0.18.14 still
@Rashida Kanchwala I updated the issue with details, I don't think it lets me x-fer it myself.
šŸ‘ 1
r
no worries, we will do it šŸ™‚ thanks Inigo!
šŸ‘ 1