Skip to content
New issue

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

[Parent] - Kedro VSCode IDE Roadmap #58

Open
21 of 33 tasks
noklam opened this issue Aug 7, 2024 · 1 comment
Open
21 of 33 tasks

[Parent] - Kedro VSCode IDE Roadmap #58

noklam opened this issue Aug 7, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@noklam
Copy link
Contributor

noklam commented Aug 7, 2024

Tracker:

0.2.0(pre-release) &0.2.2
Release after next kedro viz (1st week of September, as rely on both the main npm/python kedro-viz)

0.2.3 (bug fix release)

0.3.0

Not scheduled (wait a bit for telemetry/issues coming in):

@noklam
Copy link
Contributor Author

noklam commented Aug 21, 2024

2024-08-21 Kedro Extension / Viz integration

Flowchart Rendering

The non-deterministic rendenring of flowchart creates challenge when used with auto-reload in VSCode. Due to the high frequency of reloading, the node position will flip often for small/medium size pipeline and create an awkward UX.

  • @rashidakanchwala mention that the rendering part is done in some old code and is not easy to control because the algorithm has some randomness (as I understand it's an optimisation algorithm)
  • @merelcht challenge if it's actually required to have auto-reload

Temporary solution:

  • Disable autoreload, ask user to re-run the Run viz command instead.
    • Could have a button, but probably not worth it for temporary solution, we just keep it simple and document the behavior clearly.

Other options discussed:

  • Have an option to turn off autoreload if user want.
  • Reduce the autoreload frequency, maybe every 30s or only reload when there are new changes. This can reduce the issue but wouldn't solve the problem completely.

Vendor Viz/Ask user to install kedro-viz separately

  • @rashidakanchwala mentioned we could have a kedro-viz[core] to remove dependencies like FastAPI, uvicorn, graphql etc.
  • We could do an exercise to see how much libraries we need to vendor into the extension. @ravi-kumar-pilla gives some number showing that the dependencies may not be that big (in terms of filesize)
  • We seems to have consensus that vendoring is a better option at the end, but should do an exercise to check if it actually works. Benefit would be we can pin the kedro-viz version so future kedro-viz release will not break the extension easily, and it's easier to install as the process is invisible to the user.

Telemetry

  • @Huongg mentioned it will be useful to differentiate telemetry between a browser and extension.
  • @DimedS comment that maybe we need to have a look at kedro-viz telemetry consent flow, it should respect to the changes we made for kedro-telemetry lately, for example the environment variable DO_NOT_TRACK.
  • @rashidakanchwala mentioned that she wasn't sure if the HEAP event will be sent properly in the React component

Follow up:

  • Check with how consent flow works for kedro-viz
  • How to identify unique user/project in UI (Is it the same as kedro-telemetry?)
  • Check if the flowchart React component send HEAP event

@noklam noklam pinned this issue Aug 27, 2024
@noklam noklam changed the title [Parent] - Kedro VSCode IDE [Parent] - Kedro VSCode IDE Roadmap Sep 20, 2024
@noklam noklam added the documentation Improvements or additions to documentation label Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Todo
Development

No branches or pull requests

1 participant