Skip to content
This repository

Put the InteractiveShellApp section first in the auto-generated config files #658

Closed
fperez opened this Issue August 01, 2011 · 0 comments

2 participants

Fernando Perez Min RK
Fernando Perez
Owner

As reported on list:

Since TerminalIPythonApp and IPKernelApp inherit from the mixin InteractiveShellApp, maybe it would be more intuitive to put the InteractiveShellApp configuration before that for TerminalIPythonApp in the default, autogenerated ipython_config.py? (Also, I believe the entire section for TerminalIPythonApp appeared twice in the autogenerated file, but that's merely a cosmetic thing.)

This way, users can configure all things that affect the core Kernel class that is used both in-process and in the Qt console. This makes it much easier to configure the most commonly used pieces.

Min RK minrk referenced this issue from a commit in minrk/ipython August 01, 2011
Min RK move InteractiveShellApp before TerminalIPythonApp in TerminalApp cla…
…ss list

This is useful because InteractiveShellApp is a parent that affects
all IPython apps, so it makes sense to come first.

The change affects auto generated ipython_config.py and the output
of `ipython --help-all`.

Also prevents duplicate entries for `self.__class__`

closes gh-658
da4180d
Min RK minrk closed this August 03, 2011
Brian E. Granger ellisonbg referenced this issue from a commit January 10, 2012
Commit has since been removed from the repository and is no longer available.
Min RK minrk referenced this issue August 01, 2011
Merged

i658 #660

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.