GNOME Bugzilla – Bug 659296
Uncompatibilities with new version of IPython (at least for Fedora 16)
Last modified: 2012-02-14 18:05:37 UTC
It seems IPython console isn't working on F16 due to IPython's totally different new version. It needs to be checked.
Yes, ipython has changed the API (0.11) and some ipython plugin's code need to be changed. I'm working on this and I can describe briefly what has changed, but need more time for to see the remaining work since some functions were moved to utils module and some modules dissapeared from the API. Regards!
Hi, what is the progress on porting to the new api? I'm planing on getting ipython >= 0.11 into the next ubuntu LTS 12.04 and would like to have most rdepends fixed before doing so. A few weeks ago I attempted to fix exaile which uses pretty much the same code (as does rabbitvcs), but did not fully succeed, maybe what I have now helps: https://gist.github.com/1101425 Here the ipython wiki page for migrating, unfortunately it does not cover accercisers case very well yet: http://wiki.ipython.org/Cookbook/Updating_code_that_uses_IPython_for_0.11
Hi Julian, at this moment I'm working on this. I have a running ipython_view using the new API but still needs some work. I'm hoping in getting a working ipython plugin soon. I don't know how many modules are in the same situation, but IMHO, I think that you should keep going with the ipython >= 0.11 inclusion for 12.04. Thanks for your patch, I'll give a look and see if you solved some of the problems that now I'm having. Cheers!!
Hi, (In reply to comment #3) > [...] > Thanks for your patch, I'll give a look and see if you solved some of the > problems that now I'm having. Any update on this stuff? I just upgraded to Fedora 16 on Friday and found the same issue, then came here to report the bug and then found out this one... so now wondering about the status of it, since there seems not to be activity since one month ago. Btw, no urgency at all from my side (I have ipython 0.10 currently installed, and pinned it in yum so it doesn't ever get updated), just asking because as soon as more people upgrade their systems this could start to be a more severe issue. Also, if you think it can be of any help, I'm open to help writing the patch, but I warn you: I'm not an expert in python at all, nor in Accerciser's code either :-) Thanks!
Hi Mario! Thanks for your offering!! ;) The current status is as follows: - I reviewed Julian's patch, but this ipython_view.py seems to be an old version of the plugin :( - I tried to port the current upstream code but I'm having some problems (executing stuff) and I need to dive into ipython API, which is a little confusing for me now :S As a next step I'll talk with ipython developers for some help, I want to get this fixed before the next 3.2.x release. At this moment I haven't the code here (is in my other machine) but I'll attach it as soon as possible ;) Best regards!
Created attachment 199902 [details] first approximation Hi all! Here's a first approximation. I've commented where I'm getting problems so it has a lot of commented lines. Maybe Julian could point us some example code or a few pointers about how can I do the smooth movement to the new API. Regards!
Created attachment 199954 [details] [review] patch for above view attached a patch to that file that appearst to get most to work. But I can't seem to get thepre_readline readline startup hook to run. That would autoindent the source for multiline entries like: if 1: var=2
Hi Julian! Wooohoo! This patch is awesome! Thanks!! I've been dealing with the autoindent thing but without success :( Anyway, since I realized that the autoindent feature didn't worked before, and I really don't know how much time will take this implementation, I think that this patch is ok for being commited into the gnome 3.2.x series as a bugfix - with some slightly modifications for controlling the current exceptions -. Addingly, and based up on my testing, this patch is covering all current working functionalities and there is no regressions. More testing from a 3rd person and/or a power user here will be awesome!! - Mario ;) ;) ;) xD Distributions including ipython>0.10.* will have this plugin working like before. At this moment, this upcoming distributions with ipython>0.10.* are Fedora 16, openSUSE 12.1 and Ubuntu 12.04. Then, If we're going to fix/improve the autoindent thing and/or just add more stuff, this work could be included into the 3.3.x series without any problem and managed into separate bug reports. This week I'm very busy at work, but I'll be able to work on this during the next weekend. Thanks again, and best regards!!
Awesome! Happy to see that there's already a patch over the table to help with this issue. Thank you! (In reply to comment #8) > [...] > Anyway, since I realized that the autoindent feature didn't worked before, and > I really don't know how much time will take this implementation, I think that > this patch is ok for being commited into the gnome 3.2.x series as a bugfix - > with some slightly modifications for controlling the current exceptions -. > Addingly, and based up on my testing, this patch is covering all current > working functionalities and there is no regressions. More testing from a 3rd > person and/or a power user here will be awesome!! - Mario ;) ;) ;) xD I'm more than happy to test it, but considering what you mentioned in this paragraph, I wonder I'd better wait for the new patch with those "slightly modifications" or try this one right away. What's your take? > [...] > This week I'm very busy at work, but I'll be able to work on this during the > next weekend. Me too. Guess it's a curse we all share :-) Again, thank you! Mario
(In reply to comment #9) > > What's your take? Ok, wait for these 'slightly modifications', I'll try to get a few time this night for to attach the patch ASAP > > Again, thank you! > > Mario Thanks to you for your help here!! Regards!
for 0.12 ipython will probably deprecate a few parts of the embedded api. The parameter user_global_ns of InteractiveShellEmbed will be removed and global_ns of the __call__ function is replaced by module. In the patch here uses user_global_ns but always passes None to this parameter which as far as I understood does nothing. So you can just remove this parameter from the patch.
Created attachment 206139 [details] second approximation Hi! Sorry for the delay, but I haven't had much time for gnome in these past two months. :S So, let's continue with this ... :) As a brief resume: We had a running code, but with some itches: * autoindent doesn't work * completion works partially * some tracebacks all along the code Now (with the attached file), we have a fully working completion feature and I fixed most of tracebacks, but more testing will be appreciated. I tested it with ipython-0.11. I gave a little try with the latest stable version (0.12), but there are more problems to get solved there, the API has a lot of changes, and I need time to dive into this. Regarding the completion feature, I added a workaround for to update the "user_ns" with available pymodules from the system, because it only worked with imported modules. I hacked around the autoindent feature but without success. More help here will be great. If there aren't more problems, I'll commit these changes to the master for to get included in the next release. As I said in this thread, the autoindent feature didn't worked before porting to gi, so it can wait to be implemented. Best regards and waiting for your feedback, Javi.
(In reply to comment #12) > Created an attachment (id=206139) [details] > second approximation > [...] > If there aren't more problems, I'll commit these changes to the master for to > get included in the next release. As I said in this thread, the autoindent > feature didn't worked before porting to gi, so it can wait to be implemented. > > Best regards and waiting for your feedback, I have tested this new patch for a while this morning and, at least what it comes to the iPython view, it seems to work fine with ipython 0.11, at least with the few tests I did. However, I haven't tested it deeply yet, and that probably won't happen till next week, but I wanted to give you feedback soon anyway, so here you have it. I'll report any issue if I found any, but I guess it's safe for you to commit the change if you feel it's the right one (I haven't checked the code), since nobody but you probably knows that code better. As for me, this "just works" is already a good thing, which allowed me to upgrade from ipython 0.10 once and for all :-) Thank you!
(In reply to comment #13) Hi Mario! > > I have tested this new patch for a while this morning and, at least what it > comes to the iPython view, it seems to work fine with ipython 0.11, at least > with the few tests I did. > > However, I haven't tested it deeply yet, and that probably won't happen till > next week, but I wanted to give you feedback soon anyway, so here you have it. Thanks for your quick feedback! You rocks! > > I'll report any issue if I found any, but I guess it's safe for you to commit > the change if you feel it's the right one (I haven't checked the code), since > nobody but you probably knows that code better. This patch is ok for me too, but, I'll keep this bug opened until the patch is commited, so feel free to update it if you find any issue. > As for me, this "just works" is already a good thing, which allowed me to > upgrade from ipython 0.10 once and for all :-) The same for me! xD Again, thanks for your testing! Julian, since I tested this new code with the 0.12 version, and it needs work, we should move these next ipython_view's steps (regarding ipython>0.11 versions) into a new issue, here at gnome's bugzilla. So I'll open a new issue, putting you on CC. Your help will always be appreciated! ;) Best regards, Javi.
commit: 2745ef95d6a4f99702f3d50c8fcb3ed8c58b490e