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

is the install.py script still relevant given data_files support in setup.py? #12

Open
yifanwu opened this issue Sep 1, 2021 · 5 comments

Comments

@yifanwu
Copy link

yifanwu commented Sep 1, 2021

Hi there,

Thank you for creating this reference custom kernel!

Through some investigations, I thought that the recommended path is to use data_files support in setup.py, as opposed to manually moving files into the share/jupyter/kernels/ directory. I also noticed that the kernel.json pattern (mentioned in https://jupyter-client.readthedocs.io/en/stable/kernels.html) is not used here.

Is this current repo not up to date with the most recent ipykernel developments, or did I mis-read some instructions?

Thanks so much for the help!

@ceteri
Copy link

ceteri commented Mar 2, 2024

I agree @yifanwu as far as I can tell this no longer installs correctly. Am I missing something here, or are the Jupyter docs out of sync with the committed code and both out of sync with the demo repos?

I've tried with v1.1, using Py 3.10.11 on macOS:

IPython          : 8.22.1
ipykernel        : 6.29.3
ipywidgets       : not installed
jupyter_client   : 8.6.0
jupyter_core     : 5.7.1
jupyter_server   : 2.12.5
jupyterlab       : 4.1.2
nbclient         : 0.9.0
nbconvert        : 7.16.1
nbformat         : 5.9.2
notebook         : not installed
qtconsole        : not installed
traitlets        : 5.14.1

@blink1073
Copy link
Contributor

The echo kernel is using hatch to install the kernel, which works on PyPI and conda-forge. install.py is still there but could be removed, simplified, and absorbed into jupyter/echo_kernel@main/hatch_build.py.

@blink1073
Copy link
Contributor

blink1073 commented Mar 2, 2024

#17 is part of what we need, the other is jupyter/jupyter_client#1012

@blink1073
Copy link
Contributor

@ceteri
Copy link

ceteri commented Mar 2, 2024

thank you for the help @blink1073

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

No branches or pull requests

3 participants