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

Add load node line magic documentation #3619

Merged
merged 46 commits into from
Feb 19, 2024

Conversation

AhdraMeraliQB
Copy link
Contributor

@AhdraMeraliQB AhdraMeraliQB commented Feb 13, 2024

Description

Addresses #2011
Also addresses #2777

This page of the docs required several levels of changes, which have been addressed in this PR:

Development notes

Ignore all the early commits - this branch was created before the earlier debugging docs changes were merged into main

  • Add section on %load_node line magic
  • Add gif of using line magic
  • Add section on debugging with the line magic
  • Rejig page

Re: Rejigging the page - the following were considered:

  • The flow of the page has been adapted
  • Use of specific terminology (kernel, kernelspec) has been mostly removed. It has not been removed from the "Useful to know (for advanced users)" section as we expect those readers to be more advanced. The term "kernel" features in the section "Loading the project with kedro jupyter notebook" but is contextualised by the images provided, and as such I don't expect the terms to be as confusing. The following section "What does kedro jupyter notebook` do?" also makes references to the kernel established the in the section above it. Once again, as the use is contextualised, I don't expect the term to be as jarring.

Developer Certificate of Origin

We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a Signed-off-by line in the commit message. See our wiki for guidance.

If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.

Checklist

  • Read the contributing guidelines
  • Signed off each commit with a Developer Certificate of Origin (DCO)
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change in the RELEASE.md file
  • Added tests to cover my changes
  • Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

lrcouto and others added 30 commits January 31, 2024 01:21
Signed-off-by: lrcouto <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: lrcouto <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
@ankatiyar ankatiyar self-requested a review February 15, 2024 18:13
docs/source/faq/faq.md Outdated Show resolved Hide resolved
docs/source/development/debugging.md Outdated Show resolved Hide resolved
Signed-off-by: Ahdra Merali <[email protected]>
Copy link
Contributor

@noklam noklam left a comment

Choose a reason for hiding this comment

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

Not sure if it's covered already. I think we need to mention how to find the node_name. i.e. read from the kedro's run log if it's defined already, or go to the node definition, add a node name if needed. I think this is shown in the GIF somehow but we should give clear instruction.

docs/source/development/debugging.md Outdated Show resolved Hide resolved
Ahdra Merali added 4 commits February 19, 2024 13:26
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>
Signed-off-by: Ahdra Merali <[email protected]>

</details>

2. In your notebook, run `%load_node <name-of-failing-node>` to load the contents of the problematic node with the [`%load_node` line magic](#kedro-line-magics).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The formatting on the %load_node line magic subheading was causing issues with Sphinx when referenced. Instead, I've linked to the line magic section as a whole.

Copy link
Contributor

@noklam noklam left a comment

Choose a reason for hiding this comment

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

✨thank you!

Copy link
Contributor

@ankatiyar ankatiyar left a comment

Choose a reason for hiding this comment

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

LGTM!

@AhdraMeraliQB AhdraMeraliQB merged commit b4b1426 into main Feb 19, 2024
9 checks passed
@AhdraMeraliQB AhdraMeraliQB deleted the docs/add-load-node-line-magic-docs branch February 19, 2024 14:38
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 documentation about %load_ext kedro.ipython to make it more visible
4 participants