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

enable IPC transport for kernels #1868

Merged
merged 1 commit into from Sep 29, 2012
Merged

enable IPC transport for kernels #1868

merged 1 commit into from Sep 29, 2012

Conversation

minrk
Copy link
Member

@minrk minrk commented Jun 6, 2012

in progress, but it works with the qtconsole

Config is a bit clumsy, because the interpretation of 'ip' is actually a path when transport is IPC.

Notebook does not yet expose the option, because it's still not well integrated into the rest of the config universe.

in progress, but it works

Config is a bit clumsy, because the interpretation of 'ip' is actually a path when transport is IPC.

Notebook does not yet expose the option, because it's still not well integrated into the rest of the config universe.
@fperez
Copy link
Member

fperez commented Jun 24, 2012

Do you think we should chance this one for 0.13, or is it 0.14 material?

@minrk
Copy link
Member Author

minrk commented Jun 24, 2012

0.14, definitely

@minrk
Copy link
Member Author

minrk commented Jul 21, 2012

Test results for commit 98b82c6 merged into master
Platform: darwin

  • python2.6: OK (libraries not available: cython matplotlib oct2py pygments pymongo qt rpy2 tornado wx wx.aui)
  • python2.7: OK (libraries not available: oct2py wx wx.aui)
  • python3.2: OK (libraries not available: cython matplotlib oct2py pymongo qt rpy2 wx wx.aui)

Not available for testing:

@Carreau
Copy link
Member

Carreau commented Sep 28, 2012

ping to get status on this one.
Should we mark it as dormant, close and open an issue ?

@minrk
Copy link
Member Author

minrk commented Sep 28, 2012

I'd say it's ready for merge. The config/names can be improved, but I think that can be a separate Issue.

@Carreau
Copy link
Member

Carreau commented Sep 28, 2012

Ok, the in progress in the description was confusing me.
I'll have a look, try to re-run test_pr and merge if it passes.

@minrk
Copy link
Member Author

minrk commented Sep 28, 2012

yeah, sorry - I had intended to clean up the config API here as well (specifying ipc files with sensible path related names, instead of ip/port), but I don't have time right now. An Issue can be opened for that, if anyone cares enough about it.

@Carreau
Copy link
Member

Carreau commented Sep 28, 2012

Test results for commit 98b82c6 merged into master (ca10c42)
Platform: darwin
python2.6 : OK
python2.7 : OK
Libraries not available: azure oct2py pymongo rpy2 wx wx.aui
Extra args: ['--all']

But my computer does not find python 3... investigating.
if you could run the test on python3 just to be sure.
Thanks.

@Carreau
Copy link
Member

Carreau commented Sep 29, 2012

Cant get python3 working anymore.
Merging still.

Carreau added a commit that referenced this pull request Sep 29, 2012
enable IPC transport for kernels

works with the qtconsole

Config is a bit clumsy, because the interpretation of 'ip' is actually a path when transport is IPC.

Notebook does not yet expose the option, because it's still not well integrated into the rest of the config universe.
@Carreau Carreau merged commit af13023 into ipython:master Sep 29, 2012
@minrk minrk deleted the ipc branch March 31, 2014 23:36
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
enable IPC transport for kernels

works with the qtconsole

Config is a bit clumsy, because the interpretation of 'ip' is actually a path when transport is IPC.

Notebook does not yet expose the option, because it's still not well integrated into the rest of the config universe.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants