GNOME Bugzilla – Bug 418661
Add the accerciser Ipython console to LSR as a perk+chooser
Last modified: 2007-05-29 18:46:29 UTC
The python console in accerciser is a very valuable development tool. It would be nice if it were perk+chooser directly in LSR.
Created attachment 85534 [details] [review] first version of ipython console chooser Notes - Hotkey to open chooser is currently Alt-Shift-\ - IPython 'acc' is set to task_por.accessible at launch - IPythonPerk added to user and developer profiles. - Known issue: Chooser does not re-open after it as been closed The following exception is given. /IPythonChooser.py:80: GtkWarning: gtk_container_foreach: assertion `GTK_IS_CONTAINER (container)' failed Is this due to any weakrefs from IPythonView ?
Failed to mention, IPythonView threw the following exception: [UIRegistrar DEBUG 2007-03-29 17:17:03,847] cannot load IPythonChooser (53): 'PrintLogger' object has no attribute 'flush' giving PrintLogger a 'flush' attribute got past this exception and allowed the chooser to run.
Created attachment 85535 [details] [review] next version of ipython console chooser
As this chooser is added, keep in mind that it should only load in the developer profile. In that profile, I believe the preferred modifier is Alt-Caps instead of Alt-Shift for development related tools. Check the UI spec to be sure. / is a fine non-modifier.
Created attachment 85580 [details] [review] next version of ipython console chooser Now returning chooser reference in loadChooser exception case. This is needed to update chooser state from perk during a reload. This doesn't seem to be a problem with other choosers. Still having problems on reloading. It seems that the chooser is losing the reference to ipython_view somewhere between the end of IPythonChooser.activate and after the call to loadChooser in the perk.
I'm having second thoughts about adding this extension to the LSR package. Perhaps for the time being we should make it an extra download. It's easy enough to install with some direction. I'm just worried it's too fragile and prone to change in the near future as Eitan is still working on Accerciser.
It would also make IPython a dependency of LSR even though it will probably be rarely used.
I'm un-assigning this bug from Scott so it stops appearing as a work item. One day we should make the console available to developers, starting with ourselves. But it's really low priority right now.
Created attachment 88550 [details] all files needed for IPythonChooser
The patch is out of date: Perks/... directories have changed and LSRLog now AELog I hand edited the patch, tested it and have uploaded it. Also AEConstants/Profile.py needs a comma at the end of the line. Not in this patch as needs to be done and this is optional.
Created attachment 88794 [details] [review] Updated patch Updated patch, see previous comment in bug
I'm marking this bug WONTFIX because I don't think it should make its way into LSR as it stands. I don't want to make IPython a LSR dependency for one, and I hate to have two copies of ipython_view.py floating around between accerciser and lsr. Bug #441507 states where I'd like to head. Feel free to use this work on your own machines in the meantime.