search

jdfreder / ipython-jsobject

Exposes the JS objects directly in Python. Transparent layer that allows JS to communicate to Python and vise-versa
BSD 3-Clause "New" or "Revised" License
13 stars 1 forks source link

Bring up to date #4

Open dsblank opened 8 years ago

dsblank commented 8 years ago

@jdfreder I'm working with the Google Summer of Code group on BioJS https://biojsnet.herokuapp.com/ to integrate the biojs tools better with ipython.

@rohanjaswal2507 is the gsoc developer, and we're looking at jsobject as a starting point. Is it possible that you could help us bring jsobject up to date to work with a more modern ipython? Any pointers would help!

jdfreder commented 8 years ago

Hey @dsblank , sure, I'm glad to be of assistance. A quick peek at the code raised my attention to

There may be more hurdles, but this is a start.

dsblank commented 8 years ago

Thanks, @jdfreder ! Those may seem obvious, but would take us a while to figure out. @rohanjaswal2507 is just getting started, but it is taking a bit to understand this part of the API. Thanks again!

rohanjaswal2507 commented 8 years ago

Thanks for the help @jdfreder. I went through the code. While trying to install the code and it is causing some troubles as you mentioned. I am sharing the output on the console: http://glug.nith.ac.in:5000/4286314 Can you please help with this?

rohanjaswal2507 commented 8 years ago

@jdfreder Please ignore the last comment. The issue got fixed. However, getting some errors in the console while trying to load the notebook. Let me work on them and then will discuss here.

rohanjaswal2507 commented 8 years ago

@jdfreder The main problem I am getting while trying to run the code is that comm could not be opened. Error: Class BrowserContext not found in registry.

But, when I include a print statement in the BrowserContext class constructor(https://github.com/jdfreder/ipython-jsobject/blob/master/jsobject/jsobject.py#L57) after line 57, then the type of _comm is <ipykernel.comm.comm.Comm object at 0x7f2b045ad4d0>

So, what could be wrong with this? What can I do to ensure that comm with target_name=BrowserContext is registered?