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

Add the Stencila Node.js host for Jupyter execution context support #5

Merged
merged 4 commits into from May 11, 2018

Conversation

3 participants
@nokome
Contributor

nokome commented May 11, 2018

Thanks so much for choosing this as a project for the eLife sprint! We've worked on a couple of aspects of Stencila-Jupyter integration (file converters and a JupyterContext) but this is the first real test of them.

This PR add's the JupyterContext which allows for cells to be executed in a Jupyter Python kernel. More details in the README.

I'm going to continue work on the JupyterContext over in stencila/node to better support different mime types for execution results.

@nokome

This comment has been minimized.

Contributor

nokome commented May 11, 2018

Obligatory screenshot :)

image

@nuest

This comment has been minimized.

Collaborator

nuest commented May 11, 2018

Cool. Worldwide contributions!

@nokome: can you briefly explain or point me to the docs what the stencila host is for?

@nokome

This comment has been minimized.

Contributor

nokome commented May 11, 2018

The Host is the API "gateway" for execution contexts. It instantiates and forwards requests on to execution contexts. So for example the Host in the stencila-node package can create NodeContexts, SqliteContexts etc. It can also proxy requests on to other Hosts. So if you were to install the stencila/r package it would be able to pass on requests to create and execute in an RContext (when I fix the R package :/).

There is some draft, and probably wrong, documentation at https://github.com/stencila/website/blob/master/src/learn/computation/hosts.md

@minrk

This comment has been minimized.

Owner

minrk commented May 11, 2018

This is great, thanks!

@minrk minrk merged commit d0df78c into minrk:master May 11, 2018

@minrk

This comment has been minimized.

Owner

minrk commented May 11, 2018

Where is the pyjp key handled?

@nokome

This comment has been minimized.

Contributor

nokome commented May 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment