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

Support for bash, other magic? #51

Open
hexylena opened this issue Sep 24, 2021 · 4 comments
Open

Support for bash, other magic? #51

hexylena opened this issue Sep 24, 2021 · 4 comments
Labels
C++ enhancement New feature or request help wanted Extra attention is needed

Comments

@hexylena
Copy link

Hey y'all, I started using xeus-sql and it's fantastic, I love the interface for querying data, it feels way more jupyter-native, it's great

I was planning to use it for teaching, and I wanted to build a nice SQLite notebook which had everything the students would need, but I'm struggling with some limitations of our platform which makes it difficult to distribute more than just the notebook file. Normally I'd just add a bash magic for ! wget https://../db.sqlite at the start to do setup, but it seems those aren't implemented yet.

Is such a thing on the roadmap? I think it'd be useful to show off some database specific commands to students when needed, or run 'backup' commands.

If not, no worries, just wanted to check if that was in the plan or if I should just switch to https://github.com/catherinedevlin/ipython-sql

@marimeireles
Copy link
Member

Hey @hexylena, thank you for the kind words.
Unfortunately xeus based kernels still don't have built-in magics (with the exception of xeus-python, that's actually ipython under the hood). These magics are native from ipython and all xeus-kernels that are not using it somehow don't have it by default.
It's not on our roadmap to do something like this, something that substitutes all magics, but we would like to, if we ever get funded for it, I'd personally would love to work on it :)
Regarding your issue... I think it's not so difficult to implement something like this, and if you or anyone is interested in giving a try please say it and then I can come up with some sort of step by step on how I would approach it.
Thanks again and good luck with your classes!

@hexylena
Copy link
Author

Hi @marimeireles! Thank you for the quick response. No worries that it's not on the roadmap, oh well. It's great to hear it might still be doable though! Maybe if I find time or someone to help I'll come back and ask again for your guidance :)

@marimeireles
Copy link
Member

Hey @hexylena, no problem at all! :)
Thank you for opening the issue, raised an interesting discussion on our internal channel to me.

Yeah, maybe there are other simpler solutions to this, if you still want to use xeus-sql. You could use the terminal that's in the notebook maybe?

image
(in the inferior left side)

Anyways, I'm happy to help you or anyone that would be up for start tackling this. It's possible to only add this one command for examples, which would be of course way simpler than doing the whole thing.

@marimeireles marimeireles added enhancement New feature or request help wanted Extra attention is needed C++ labels May 23, 2023
@marimeireles
Copy link
Member

marimeireles commented May 23, 2023

How hard is it to implement support for terminal cmds @martinRenou ? You did it for xeus-python or did you get it for free smh?
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants