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

Higher level API #4

Closed
n-riesco opened this issue May 22, 2015 · 3 comments
Closed

Higher level API #4

n-riesco opened this issue May 22, 2015 · 3 comments

Comments

@n-riesco
Copy link
Owner

From #2 (comment) by @willwhitney:

"
My ideal situation would be to have jmp implement 100% of the communication between a kernel and a client, heartbeat included. The project using jmp would then not need to know anything about the Jupyter protocol at all.

Instead, jmp could expose an event that could be subscribed to with something like

jmp.on('disconnected', function() {})

Really, the overall API I'm proposing would be higher-level than what's in jmp right now. The dream would be

jmp.on('execute_request', function(requestObject) {})
jmp.on('complete_request', function(requestObject) {})
jmp.on('inspect_request', function(requestObject) {})

etc.

But maybe that's a conversation for a different day — this is already a great start.
"

@n-riesco
Copy link
Owner Author

From #6 (comment) by @rgbkrk

"[...] I'd eventually see us in a position of providing a Session object. We three have all implemented it in separate ways within ijavascript, Hydrogen, and sidecar. During SciPy when I'm hacking with folks on some other electron prototypes, we'll certainly be doing the same again."

@n-riesco n-riesco changed the title HIgher level API Higher level API Jul 16, 2015
@rgbkrk
Copy link
Contributor

rgbkrk commented Feb 9, 2016

Just to leave a note on this, I'm now running with a higher level API on top of Observables. This is no longer a requested feature from me, we'll just build on top of jmp and make new packages.

@n-riesco
Copy link
Owner Author

n-riesco commented Feb 9, 2016

OK, I will close this issue now. I imagine we will revisit this issue when Hydrogen's kernel manager gets redesigned.

@n-riesco n-riesco closed this as completed Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants