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

Discuss: Remove minimal kernel #213

Merged
merged 1 commit into from
May 27, 2016
Merged

Conversation

parente
Copy link
Member

@parente parente commented May 27, 2016

The minimal-kernel image sees very little use (less than 600 pulls since it was added). It's purpose was to get the kernel gateway (KG) into a small image that could be the basis for adding kernels and libraries for specific use cases. The work over in the incubator on the dashboard projects use the KG heavily, but we quickly found that adding it to any of the other fully-featured stack images (all-spark-notebook, datascience-notebook, ...) was always easier than trying to build up what was needed from a small image.

Anyone object to killing off this stack?

@rgbkrk
Copy link
Member

rgbkrk commented May 27, 2016

No objection here. It's too bad we can't see when people have pulled images, over time.

@parente
Copy link
Member Author

parente commented May 27, 2016

I thought about turning https://gist.github.com/parente/316d5c242aeb484484c8 into something that collects the stats once a day over time like https://gist.github.com/parente/facb555dfbae28e817e0 but haven't gotten around to it.

@rgbkrk
Copy link
Member

rgbkrk commented May 27, 2016

Well done! Where have you been storing these notebook count stats? Just updating the CSV in the gist?

@parente
Copy link
Member Author

parente commented May 27, 2016

I have a (mostly) scheduled task to clone the gist, nbconvert execute the notebook, commit and push the updated files back to the gist.

@minrk
Copy link
Member

minrk commented May 27, 2016

Seems fine to me. Thanks, @parente!

(c) Copyright IBM Corp. 2016
@parente parente force-pushed the remove-minimal-kernel branch from 6387d29 to 48cf895 Compare May 27, 2016 11:22
@parente
Copy link
Member Author

parente commented May 27, 2016

Rebased on master after sending in #209.

@rgbkrk rgbkrk merged commit 45f83f4 into jupyter:master May 27, 2016
@jakirkham
Copy link
Member

I'd always thought about using this, but never ended up doing so. Sounds reasonable to get rid of it to me too.

@rgbkrk
Copy link
Member

rgbkrk commented May 27, 2016

Thank you for deletion! You even cleaned up the diagrams which is super stellar.

@parente parente deleted the remove-minimal-kernel branch March 26, 2018 01:20
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.

4 participants