-
Notifications
You must be signed in to change notification settings - Fork 903
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 documentation about %load_ext kedro.ipython
to make it more visible
#2777
Comments
%load_ext kedro.ipython
more visible%load_ext kedro.ipython
to make it more visible
I almost open this issue again 🙃 Another user got confused because they couldn't really use |
For context, kedro/kedro/framework/cli/jupyter.py Lines 90 to 91 in c3c93cb
|
@noklam mentions that |
I'm happy to do this but need more steer about (a) the problem and (b) the location in docs that we include it. |
💯 %
The problem of the latter two is that they're too rigid:
So I think |
Another user whose problems with |
Almost duplicate the issue. I agree with everything said here. Ideally the Kedro specific kernel is a nice idea, in practices it suffers from different issue.
|
Another user who could have benefitted from this https://linen-slack.kedro.org/t/16320811/i-just-learned-that-the-command-kedro-jupyter-notebook-can-b#435be7c7-e8cd-4791-b6ca-37637fd8b194 |
yay ❤️ |
Description
We should make
%load_ext kedro.ipython
more visible, since it's tool-independent (as long as it's IPython-compatible) and also independent on how the service was launched (think for instance Databricks notebooks, Google Colab, and VSCode).Context
A user wanted to use the Kedro extension on VSCode and didn't find a way in the docs https://www.linen.dev/s/kedro/t/13090672/hi-everyone-is-there-some-setting-that-could-allow-for-kedro#30d9d3e7-73e3-4719-8be5-882a947088e4 cc @m-gris
even if it's there already https://docs.kedro.org/en/stable/notebooks_and_ipython/kedro_and_notebooks.html#managed-services.
In fact, I don't even use
kedro jupyter notebook
that much 🤔 always resort to%load_ext kedro.ipython
everywhere.Possible Implementation
We should probably restructure those pages a bit, so that it's clear that the command can be used in many places, and not just "managed services".
Possible Alternatives
I didn't think any.
The text was updated successfully, but these errors were encountered: