-
Notifications
You must be signed in to change notification settings - Fork 21
Resolved #107 -- Added support for line & cell magics by subclassing MetaKernel #109
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
base: master
Are you sure you want to change the base?
Conversation
dwijy-piggy
commented
May 23, 2025
- Subclassed MetaKernel to implement custom line and cell magic commands
- Updated output methods to follow MetaKernel’s conventions
- Adjusted execution logic to align with MetaKernel's do_execute function
Thanks for the PR. You can add the MetaKernel requirement at: dyalog-jupyter-kernel/setup.py Lines 23 to 25 in ef152d3
|
80342d9
to
f3cebc6
Compare
done |
Try squashing the remove import commit: https://youtu.be/42392W7SgnE?t=334 |
f3cebc6
to
a402e3d
Compare
done |
There's an extra commit by @martanit. You can delete that commit in |
…ssing MetaKernel - Subclassed MetaKernel to implement custom line and cell magic commands - Updated output methods to follow MetaKernel’s conventions - Adjusted execution logic to align with MetaKernel's do_execute function
a402e3d
to
719b48d
Compare
done |
done |
bdbe457
to
b26bc21
Compare
My notebook hangs when I try to run a cell. Is there a working demo I can try? `jupyter notebook` output
|
Entering the following commands into Terminal on macOS successfully set up the kernel for me:
If you are using just a kernel.py and kernel.json file to test this locally, the error might occur if you aren't starting the kernel using |
Still the same problem. I'm on Linux (NixOS). However it works when I tried it on Windows. |
Could you send the output from this command: |
The main branch displays the same `jupyter console --kernel=dyalog_apl --debug`
|
@xpqz can you see if it works on your end? Thanks |
Works for me on Windows 👍! |
It would be good to see an outline of the consequences of switching from ipykernel.kernelbase to MetaKernel as the baseclass. Whilst convenient, are there any implications or future limitations we should be aware of? |