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

Kernel crashes on attempt to tab complete #5

Closed
itkach opened this issue Mar 2, 2015 · 2 comments · Fixed by #7
Closed

Kernel crashes on attempt to tab complete #5

itkach opened this issue Mar 2, 2015 · 2 comments · Fixed by #7
Labels

Comments

@itkach
Copy link

itkach commented Mar 2, 2015

Type func in a cell, hit Tab, observe "Dead kernel" status label appears and the following exception in ijs output:

[I 13:36:23.062 NotebookApp] KernelRestarter: restarting kernel (1/5)
WARNING:root:kernel be7eec2c-4bf4-4f2e-95e3-2a4de794abe6 restarted
events.js:75
      throw er; // Unhandled 'error' event
            ^
TypeError: Cannot read property 'slice' of undefined
    at parseExpression (/home/itkach/lib/node_modules/ijavascript/lib/kernel.js:291:30)
    at Kernel.complete_request (/home/itkach/lib/node_modules/ijavascript/lib/kernel.js:369:22)
    at Kernel._onMessage (/home/itkach/lib/node_modules/ijavascript/lib/kernel.js:109:33)
    at emit (events.js:108:17)
    at Socket._flush (/home/itkach/lib/node_modules/ijavascript/node_modules/zmq/lib/index.js:510:19)
    at _zmq.onReady (/home/itkach/lib/node_modules/ijavascript/node_modules/zmq/lib/index.js:193:12)

iojs v1.1.0, ijavascript 4.1.4, ipython 3.0.0

@n-riesco
Copy link
Owner

n-riesco commented Mar 3, 2015

Thank you for reporting this ussue. I've tracked down this issue to ipython 3.0.0 using protocol version 5.0 despite ijavascript reporting the use of version 4.1. I will provide a solution soon.

@itkach
Copy link
Author

itkach commented Mar 3, 2015

Indeed, tab complete doesn't crash in IPython 2.2.0

n-riesco added a commit that referenced this issue Mar 3, 2015
kernel.js: send busy/idle status messages

* Fixes issue #5
n-riesco added a commit that referenced this issue Mar 4, 2015
jp.js: Fix wire protocol implementation

* Closes #5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants