ipython / ipykernel

IPython Kernel for Jupyter
https://ipykernel.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
649 stars 367 forks source link

Proposal: remove some ipyparallel-specific functionality #17

Open minrk opened 9 years ago

minrk commented 9 years ago

Mainly the apply message-related bits. These aren't part of the wider Jupyter message protocol, and are wholly Python specific. These can be moved to ipyparallel, and attached to the Kernel in a subclass.

takluyver commented 9 years ago

No objections from me.

minrk commented 9 years ago

Missed the boat for 4.0 on this one, but I still think it's the right move in the long run.

dsblank commented 9 years ago

FYI, we use the ipyparallel bits in wrapper kernels (via metakernel) to run other languages in parallel. This is quite powerful, and hope that we can keep that functionality.

minrk commented 9 years ago

@dsblank how do you use those kernels in parallel? Are you using ipyparallel at all? I mainly mean to remove the native-serialization stuff (data-pub, apply), not anything else, which shouldn't affect any kernels other than IPython.

minrk commented 9 years ago

@dsblank I've implemented most of what I intended in ipython/ipyparallel#61. I don't understand enough about how your use of parallel in metakernel works, so I don't know how to tell what would be affected. Can you provide some examples that I can test with?

dsblank commented 9 years ago

Sorry for the delay. Here is a short example:

  1. Install a metakernel-based kernel: pip3 install metakernel_bash --user
  2. Start a cluster
  3. Start the kernel
  4. Start remote kernels: %parallel metakernel_bash MetaKernelBash
  5. Execute a command: %px uname

Hopefully that will still work. Thanks!

minrk commented 9 years ago

@dsblank thanks, that still works after the above PR. Since it looks like you are sitting on top of the existing IPython engines and clients, moving the implementation shouldn't have any effect on your use case.

dsblank commented 9 years ago

Thanks for checking @minrk ! It is quite nice to have Parallel Java, Parallel Scheme, Parallel Cobol ...

Carreau commented 6 years ago

bump to 6.0 ?