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

Improve Kedro-viz experience on notebooks #1722

Merged
merged 4 commits into from
Feb 5, 2024
Merged

Conversation

SajidAlamQB
Copy link
Contributor

@SajidAlamQB SajidAlamQB commented Jan 29, 2024

Description

When using the %run_viz command for kedro-viz in Jupyter notebooks, it should open Kedro-viz in a new browser tab.

Development notes

image

When clicking Open Kedro-Viz a new tab is opened:

image

QA notes

Added one new unit tests that checks the target attribute is set to _blank, and the link text is Open Kedro-Viz.

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added new entries to the RELEASE.md file
  • Added tests to cover my changes

@SajidAlamQB SajidAlamQB self-assigned this Jan 29, 2024
Signed-off-by: Sajid Alam <[email protected]>
@SajidAlamQB SajidAlamQB marked this pull request as ready for review January 30, 2024 14:30
@SajidAlamQB SajidAlamQB linked an issue Jan 30, 2024 that may be closed by this pull request
1 task
@SajidAlamQB SajidAlamQB requested review from jitu5 and NeroOkwa January 30, 2024 14:35
@astrojuanlu
Copy link
Member

I notice in the screenshot that you're using an old version of Jupyter notebook, @SajidAlamQB could you try with notebook>=7.0? It was completely refactored and it's now based on JupyterLab internally.

@SajidAlamQB
Copy link
Contributor Author

I notice in the screenshot that you're using an old version of Jupyter notebook, @SajidAlamQB could you try with notebook>=7.0? It was completely refactored and it's now based on JupyterLab internally.

Here is what it looks like with notebook>=7.0:

image

image

Copy link
Contributor

@rashidakanchwala rashidakanchwala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look's great. Thanks Sajid! :)

Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 Should this change be included in the release notes?

Signed-off-by: Sajid Alam <[email protected]>
@SajidAlamQB SajidAlamQB merged commit 882f35c into main Feb 5, 2024
17 checks passed
@SajidAlamQB SajidAlamQB deleted the improve-notebook-exp branch February 5, 2024 12:27
santana98 pushed a commit to santana98/kedro-viz that referenced this pull request Feb 19, 2024
* Add link to open in new tab

* add test and lint

Signed-off-by: Sajid Alam <[email protected]>

* Update RELEASE.md

Signed-off-by: Sajid Alam <[email protected]>

---------

Signed-off-by: Sajid Alam <[email protected]>
Signed-off-by: Elias Santana <[email protected]>
@ravi-kumar-pilla ravi-kumar-pilla mentioned this pull request Mar 1, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Kedro-viz experience on notebooks.
4 participants