-
Notifications
You must be signed in to change notification settings - Fork 283
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
ICSHARP kernel does not work with Jupyter 4.0 #70
Comments
And from list was working on 3.0, seem to be a change in the adapter, where if "version" is in header, but it empty string, is not assumed to be "4.0" |
That's very strange. A message with an empty version string has never been valid, so if it is the case, the messages valid in any version of the protocol, and adaptation should either fail outright or be undefined. What else is there to do? |
Figure out what the regression is between 3.0 and 4.0, as far as I understood from the above issues and on ML it works on 3.0 so there is no reason not to work on 4.0. |
@Carreau I don't think there's any change between IPython 3 and Jupyter 4. I don't see any reference to this working in IPython 3, only IPython 2, prior to the msgspec change. Since IC# puts an invalid version field in the header, adaptation fails to detect the correct version. Fixing this header (zabirauf/icsharp#7) ought to allow adaptation to start working. I don't think there are any changes to make, here. |
I might have just misunderstood the ML conversation, I was under the impression that it was working on 3.0 |
@minrk @Carreau with the latest changes the kernel is still not working on ipython 3.0/4.0, but works on ipython 2.0. Bunch of null arguments in zmq connection messages:
|
Closed by zabirauf/icsharp#10 apparently ! Thanks ! |
zabirauf/icsharp#5
zabirauf/icsharp#4
The text was updated successfully, but these errors were encountered: