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

Callysto menu extension #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AyushChaudhari2001
Copy link

Added 2 new extensions:

Banners - Automatically creates Callysto banners in new Jupyter notebooks
Menu - Provides options to create predefined cells with code snippets for commonly used libraries

@ianabc
Copy link
Member

ianabc commented Jul 6, 2023

Hi Ayush, thanks for the PR! So far I've tried to avoid adding the jupyter_contrib_extensions module (and jupyter_nbextensions_configurator), would it be practical to rewrite these to stand alone? Or are they closely tied to those projects?

@AyushChaudhari2001
Copy link
Author

Hi @ianabc, yeah the extensions I've made are pretty closely tied to those modules so I'm not sure how practical it would be to rewrite the code. Out of curiosity, is there a reason for trying to avoid using those modules?

@ianabc
Copy link
Member

ianabc commented Jul 7, 2023

I forget, but I think it was getting difficult to untie some of the support requests for the extensions from other parts of the infrastructure.

@AyushChaudhari2001
Copy link
Author

@ianabc Is there any way I can test it out on the Callysto infrastructure? If not maybe I'll look at the code you wrote and try to rewrite mine to not be reliant on the modules

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.

2 participants