=== lifeless_ is now known as lifeless === spm` is now known as spm === SpamapS_ is now known as SpamapS === nhaines__ is now known as nhaines === wgrant_ is now known as wgrant === AlanChicken is now known as AlanBell === ferai is now known as jefferai [08:00] mandel: hey, im going to start working at 9, 8 is just too early [08:01] fagan: sure, you really dont have to start when I do, follow the normal office hours [08:01] fagan: one quick question, did you finish the last exercise? [08:02] is to plan the next one or not [08:05] mandel: well it needs about 20 more lines of code for the window title stuff [08:05] mandel: so you can get another one ready and ill get to it when I finish the othuer [08:05] *other [08:06] fagan: I'll get the next one ready, but we wont move to it until you get a pass for the last one either from ralsina or from me [08:07] mandel: yeah sure [08:11] * mandel grabs coffee [09:03] * fagan working [09:34] mandel: im confused im trying to set the window title and im getting AttributeError: 'Ui_Dialog' object has no attribute 'setWindowTitle' [09:34] fagan: what class does Ui_Dialog inherit from? [09:35] the window class? [09:35] fagan: have you looked at the generated code? [09:36] ah I fixed it [09:36] fagan: what was wrong? [09:37] mandel: I had it addressing the dialog and I should have been addressing the object [09:38] fagan: can you pastebin the code you have so far, I dont think I understand you [09:39] mandel: I did self.ui.setWindowTitle when it should have been self.setWindowTitle [09:40] yes, exactly, but what I dont understand is you sentence, 'I had it addressing the dialog and I should have been addressing the object' [09:40] that sentece does not make sense [09:43] mandel: self is the QDialog object and the ui bit is the py where most of the object is defined so when I did self.ui I was addressing the .py class that doesnt extend QDialog so it doesnt have setWindowTitle defined but the QDialog does so self works but self.ui doesnt since its not defined there and doesnt extend the class [09:43] * fagan cant really expalin things too well at times [09:44] It works thats the main thing [09:44] And I understand what was wrong [09:44] fagan: so what is wrong wit the initial sentence [09:45] mandel: I was trying to shorten what I was saying in the hope that it was understandable [09:45] fagan: just working when you are learning is not enough, you have to understand what you are doing, trial an error when programming is not a good technique, specially with a lang that is not compiled [09:46] fagan: I know what you were trying to say, I knew the issue, that is why I asked you too look at the parent class of the generated code, now again, wit ht einitial setence, why is it wrong [09:46] read carefully, and understand what you were saying and what you wanted to say [09:53] fagan: ^? [09:56] mandel: Yeah I said it in the wrong way [09:57] fagan: ok :) [10:03] mandel: lp:~shanepatrickfagan/+junk/string_list [10:03] actually hold on forgot to check pep8 and pylint [10:03] * mandel looks [10:04] fagan: why dont you write a script that runs them before you can do the push, that way you will always do them without having to do it by hand [10:05] mandel: or just write a note that says "DONT FORGET TO CHECK PEP8 AND PYLINT" [10:09] fagan: I think mandel suggestion is a lot more professional [10:15] fagan: ping me whenever you are ready for the review [10:15] mandel: pylint fails but not on anything big so ill just disable them if thats ok [10:17] fagan: sure, I'll take a look at the disable warnings [10:19] mandel: well 2 are about var names, 1 is about modifying a var that has outer scope internall [10:19] *internally [10:19] that kind of thing [10:20] fagan: the var names is not a bg deal, but scopes should be taken care of [10:20] mandel: well its at class level but its still internal [10:21] fagan: push the code and I'll take a look [10:21] kk [10:21] is the easiest :) [10:22] mandel: oh I should make my quit button work [10:22] so 1 sec [10:30] mandel: ok good except pylint [10:30] so pushed [10:32] * mandel looks [10:34] lts of bugs but it works [10:38] fagan: I get the following when I try to add a string: [10:38] Traceback (most recent call last): [10:38] File "main.py", line 36, in add_dialog_show [10:38] self.ui_.list_widget.emit(SIGNAL("accept(self)"), self) [10:38] NameError: global name 'SIGNAL' is not defined [10:38] mandel: oh I removed QtCore from the import list [10:39] fagan: also, I done thing the emit is correct, 'self' is not something that Qt understands [10:39] fagan: also, you could have used the new PyQt syntax to emit the signal, which would have avoided the import error [10:39] mandel: ah im doing it the old way again [10:40] fagan: also, when I'm editing an item I get the following: Traceback (most recent call last): [10:40] File "main.py", line 49, in edit_current_item [10:40] self, 'Edit' + list_type, "Enter" + self.list_type + ':') [10:40] NameError: global name 'list_type' is not defined [10:41] fagan: up and down are meant to move the items, right? [10:42] mandel: I thought it was to move the selected item up and down [10:42] I can change it though [10:42] fagan: ok, not a huge deal, but sort does not work [10:42] mandel: that turns on auto sorting [10:42] its a toggle [10:43] fagan: I see banna infront of apple all the time [10:44] mandel: then there must be something wrong with how im checking if the toggle is checked or not [10:44] mandel ping [10:44] ralsina_: pong [10:44] hi everyone, just popping in 15 minutes before leaving for sprint. [10:45] mandel: what version mismatch? [10:45] ralsina_: oh is that the ar sprint [10:46] ralsina_: it seems that I have installed a higher version of Qt than the one used by pyqt and py2exe does some funny bundling together [10:46] fagan: yes [10:46] mandel: ugh [10:46] ralsina_: it adds the systems Qt dlls, and pyQt does not like that [10:46] ok, I was curious, because buildint PyQt from source on windows is a beast [10:47] ralsina_: yes, I've noticed… I'm going to try and downgrade the Qt version in my system [10:47] if I find the download... [10:47] what version do you need? [10:49] ralsina_: let me take a look [10:50] * fagan needs milk for tea brb [10:52] ralsina_: do you remember the function used to get the qt version out of qt [10:52] it was something like get_qt_version... [10:53] ralsina_: got it, is qVersion... [10:53] one sec [10:55] ralsina_: I need 4.7.1 while the current Qt version I have in the system is 4.7.0 [10:55] so I need the extra .1 Although I'm not sure that wil solve the issue [10:56] hmmm... so what you really need is upgrading Qt? that's easier! [10:56] ralsina_: yes, sorry, I'm confused because the SDK offers 4.7.0 and the libs only offers 4.7.2 [10:56] fuck… I hate windows [10:57] mandel: get Qt 4.7.2 and PyQt 4.8.3 (I think that one is 4.7.2 based) [10:57] ralsina_: I'm downloding it atm [10:57] mandel: ok [10:58] gotta run to the sprint now [10:58] if one has two weks of sprints, is that still a sprint, or is it some sort of medium distance event, like the 800 meters dash? [10:59] mandel: you on sprint also ? [10:59] czajkowski: nah, the bastards left me behind.. windows guy… mine is next week [11:00] I blame ralsina_ for making me work on windows!!! [11:00] mandel: right because coming to a sprint in BA and leaving for London would have been so much fun ;-) [11:01] Anyway, good luck with that! [11:02] mandel: lanauge, still an Ubuntu channel :) [11:09] ralsina_: hehe, have fun in the sprint [11:10] czajkowski: bastards is not that bad… Is widely used in Manchester to refer to friends :P [11:19] * fagan back [11:19] sorry bout that needed milk [11:22] oh and mandel the language in manchester is like a subset of english [11:23] its not exactly the one to follow :) [11:23] fagan: hehe, I know, is an interesting one :) [11:23] fagan: can you take a look at the errors I mentioned and ping me when done? [11:25] mandel: yep looking into them now [11:25] trying to translate the old style singal emiting to the new style [11:28] 'lo #ubuntuone [11:29] hellow duanedesign [11:29] mandel: lol, "bastards is not that bad… Is widely used in Manchester to refer to friends" [11:29] hehe spelling ftw [11:29] fagan: oh i am the worst [11:30] duanedesign: have you read my blog :P [11:33] * duanedesign thinks it is in his RSS Reader. [11:33] there are some nice spelling mistakes if I dont actually look over the posts there [11:34] * fagan loves that he spell checks his code otherwise it would be a big problem :) [11:34] fagan: shame you dont do that to your blog posts [11:35] fagan: where is my spelling error? [11:35] fagan: I think i added your blog awhile back to follow your zeitgeist posts [11:35] * mandel curious .... [11:35] mandel: what spelling mistake? [11:37] you said: 'hehe spelling ftw' [11:37] mandel: I said hellow [11:37] :) [11:37] ah, ok :P [11:37] I though it was me and I was very puzzled :) [11:47] mandel: Im having a little problem with the new signals thingy [11:48] what is the issue? [11:48] mandel: http://paste.ubuntu.com/589208/ this should work shouldnt it? [11:49] its giving me an error saying that accept doesnt have any attribute emit [11:54] mandel: did you get that? [11:55] fagan: sorry, the irc client went nuts, can you repeat it? [11:55] mandel: http://paste.ubuntu.com/589208/ this should work shouldnt it? [11:56] its giving me an error saying that accept doesnt have any attribute emit [11:56] fagan: can you paste the full code please [11:56] ok [11:57] fagan: in theory, it should work, maybe there is something else going on [11:57] http://paste.ubuntu.com/589211/ [11:59] * mandel looks [11:59] mandel: its in the add bit [12:01] fagan: signals have to be added as a class instance, not a local var, there is some magic done by pyqt at the metaclass level [12:01] fagan: move the definition to accept to be a class instance and that will work [12:02] mandel: cool [12:03] fagan: it does make sense that a sugnal has to be owned by the class and not by an instance of the class, right? [12:04] mandel: well it makes a lot of sense since it has functions attached [12:05] and you could probably overload stuff too [12:06] fagan: well, I'm more thinking of the fact that the signals have to be own by all the instances, otherwise, you could find an instance of a class that has a diff contract that the others, right? [12:07] mandel: yeah [12:07] mandel: well it makes a lot of sense since it has functions attached [12:08] damn keyboard [12:08] so shared custom signals around the program [12:08] * fagan didnt mean to repeat that but still getting used to the smaller keyboard [12:11] works now [12:11] good === beuno is now known as beuno-lunch [12:15] im going to take a quick lunch and finish off that last little bit when I get back [12:16] the signal is working now though so its almost done now [12:44] ralsina_: ping [12:45] * fagan back [12:46] mandel: I think the only thing im missing now is the sort not turning on [12:46] so ill go at that [12:46] ok, I need to go and walk the dog, I'll be back later [12:46] cool should be done by then [12:48] fagan: perfect then [12:48] ralsina_: FYI: I'm stupid, check my twitter :P === beuno-lunch is now known as beuno [13:00] karni, hi. Of my review of GreenDroid branch. Fix it in the one GreenDroid branch, and "bzr merge" that to the other branch that is based on GreenDroid branch. [13:00] CardinalFang: ACK will do that [13:01] karni, there is a bzr plugin to support this work scheme. http://wiki.bazaar.canonical.com/BzrPipeline [13:01] karni, but, if you're careful, you need that only rarely. [13:02] CardinalFang: Thanks!! [13:21] CardinalFang: Reassure me where we're at. u1f-greendroid has "pending" review, u1f-extract-login (fork of u1f-greendroid) has been reviewed by you. Maybe I could get away with it this once and only merge the u1f-extract-login branch to the trunk? [13:22] CardinalFang: unless I didn't get the " Of my review of GreenDroid branch" right and you were going back to review u1f-greendroid === bac` is now known as bac === ehw_ is now known as ehw [13:30] done and tested \0/ [13:31] * fagan has a victory cookie [13:40] mandel: when you get back give it a try [13:44] alecu: would you please do a trivial review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/translate-volumes/+merge/55989 ? I already alerted dpm about the new translatable string [13:44] nessita, sure [13:45] thanks! [13:45] nessita, here, have some more pañuelos [13:47] nessita, approved. [13:47] alecu: thanks! [13:50] CardinalFang: ignore my last question, I'll handle it. [13:53] morning Chipaca [13:54] mo'in' [13:54] (or afternoon) [14:02] * mandel back [14:02] nessita: we dont have a stand up, or do we? [14:02] I mean, you are all over there... [14:02] fagan: ok, I'm looking [14:03] mandel: I changed the code for sorting around to just do a static sort rather than the toggle what I had [14:03] but at least I know how to use toggle now so it wasnt a bad thing to learn then [14:04] fagan: well, it is a learning experience, which is good :) [14:04] mandel: yeah [14:04] fagan: can you remind me the branch? [14:05] lp:~shanepatrickfagan/+junk/string_list [14:05] superb, thx [14:08] fagan: why do you need the accept signal? who is listening for it? [14:08] mandel: it was in the task [14:09] * fagan doesnt have the link today but it was in there [14:09] ok, let me find it [14:09] one sec [14:13] mandel: I don't think so [14:13] nessita: ok :) [14:13] fagan: the exercise was the following: http://paste.ubuntu.com/589260/ [14:13] right? [14:13] mandel: yep [14:14] fagan: I think that when the signal has to be raised is not clear, so I wont complain at all about that [14:15] mandel: cool [14:15] fagan: but your signals is missing a parameter, right? === kenvandine_ is now known as kenvandine [14:15] yeah I can add it now though [14:16] fagan: also, just for compliteness, take a loog at http://doc.qt.nokia.com/4.7/qdialog.html#accept [14:16] which is what I was looking for regarding the reject and the accept methods [14:17] mandel: so do that rather than than listening for the signal and writing my own methods [14:18] lots of netsplits today [14:18] fagan: yes, I think it is nicer to use the actual api from Qt in this case [14:18] mandel: want me to change it? [14:19] fagan: also on last thing, the dialog to edit the fruit are missing a space and the adding and the editing dialgos have the same title, can you fix that? [14:19] sure [14:19] fagan: yes, use the accept from QDialog, it makes more sense, right? [14:19] yeah [14:19] * fagan makes tea first :) [14:19] fagan: cool, let me know when you are ready for the next one then :) [14:26] ralsina_: ping [14:26] mandel: pong [14:27] ralsina_: what do we do with https://code.launchpad.net/~mandel/ubuntu-sso-client/windows_ui_3/+merge/55561? [14:27] ralsina_: also, did you understand what was my issue with PyQt and PySide? [14:36] mandel: could you explain the done() thingy for me [14:37] I cant seem to figure it out by messing with the code [14:38] fagan: can you please show me the code you are using? [14:38] mandel: if it fixed itself by removing pyside, I don't need to understand it ;-) === txwikinger2 is now known as txwikinger [14:39] mandel: that branch you need another review? I can ask someone, I have 5 u1 people within striking distance ;_0 [14:39] mandel: I dont really think that would be helpful since its so wrong [14:39] fagan: i would be, It be nice to see what you are thinking [14:39] ralsina_: if you do get an extra +1 it would be great [14:40] mandel: sure, I'll just steal chad's notebook when he's not looking [14:41] ralsina_: superb :) [14:41] CardinalFang: are you in BA? [14:42] http://paste.ubuntu.com/589267/ [14:42] mandel: ^ [14:43] mandel, yes. ralsina_ is <- . thisfred is ->. [14:43] BA? [14:44] Buenos Aires .ar [14:44] ah [14:44] AR would have made more sense to me :) [14:44] * fagan thought it was the airline BA [14:44] CardinalFang: ah, lucky you, I though you were in a loan to mobile... [14:45] mandel, I was until Friday. [14:45] (any excuse to go on a free trip to AR) [14:49] mandel: so did you look at it ^ [14:49] fagan: the done action is not for you to be use with the InputDialogs, but for you to execute when you wanna close your dialog, look at what the docs say [14:49] Closes the dialog and sets its result code to r. If this dialog is shown with http://doc.qt.nokia.com/4.7/qdialog.html#exec(), done() causes the local event loop to finish, and http://doc.qt.nokia.com/4.7/qdialog.html#exec() to return r. [14:49] mandel: ah ok [14:50] fagan: you have to be more careful when you read the docs [14:50] * fagan miss understood what you wanted with the done bit [14:52] I really love bzr at times :) === rodrigo__ is now known as rodrigo_ [14:54] fagan, well, on some of these airplanes, one has to help pedal. === wgrant_ is now known as wgrant [14:55] reboot. brb. [14:57] mandel: done [14:58] with both things [14:58] ok, smae branch, right? [14:58] yep [14:58] I made the dialog text better and did the quitting with self.done [14:58] and I tested it [14:59] fagan: I get the following: [14:59] Traceback (most recent call last): [14:59] File "main.py", line 50, in edit_current_item [14:59] current.setText(text) [14:59] AttributeError: 'NoneType' object has no attribute 'setText' [15:00] fagan: to reproduce do the following [15:00] open the app, click directly on edit without clicking in an item [15:00] mandel: ah ok I can fix it [15:01] I missed that error when I tested it [15:02] edit_button.enabled = (current != None) [15:03] Jesus fsck, I overdid it.. I had unmerged improvements to the database, and now I'm all spaghetti with the branches. [15:03] fagan: also, while you are at it, you have missing all the keyboard accelerators [15:03] * karni recovers [15:03] karni, :) :( [15:04] CardinalFang: I'll learn it the hard way! [15:04] karni: bolognese? [15:04] mandel: yeah [15:04] ouch [15:04] mandel: the keyboard is working for me are you sure? [15:04] karni, everyone does this at least once. Now, you'll be scared to branch for a while. :) [15:04] CardinalFang: yeah, I am already! [15:05] * karni is scared of unmerged branches now [15:05] I somehow knew I would end up like this ehh. Will be more careful next time. [15:06] karni, I'm going to make a tool that lets me separate a single tree's multiple changes into several discrete commits. [15:07] fagan: according to the example ui, if I press ctrl+e a should be able to edit, right? [15:07] CardinalFang: neat [15:07] mandel: it should be alt+e [15:07] CardinalFang: I'd love that! [15:07] fagan: oh, let me try, mea culpa [15:07] karni, mandel, maybe on a Crazy Friday. [15:08] fagan: you are right, they are there [15:09] mandel: and the student becomes the master :P [15:09] CardinalFang: yeah, that friday idea is great… I've got more ideas than fridays :( [15:10] mandel: and != None doesnt work on the currentItem for some reason [15:10] mandel: and != None doesnt work on the currentItem for some reason [15:10] I tried switching to currentRow either but that is returning 0 when 0 means nothing and the first row [15:11] so I dont know how to fix it other than editing the 0 row by default [15:12] nessita: bug #750309 [15:12] Launchpad bug 750309 in ubuntuone-control-panel "Services tab layout doesn't work right (affects: 1) (heat: 6)" [Undecided,New] https://launchpad.net/bugs/750309 [15:13] fagan: may I see the codE? [15:13] mandel: sure [15:15] mandel: http://paste.ubuntu.com/589281/ [15:17] CardinalFang: haha, our setup script is vertero'k-prone: wget: unable to resolve host address `verterok.com.ar' [15:18] karni, ^ a reason I wanted releases on launchpad. :( [15:19] The southern hemisphere is a single point of failure. [15:19] verterok`: is there any chane you make a review for me? https://code.launchpad.net/~nataliabidart/ubuntuone-client/disable-properly/+merge/55994 [15:19] fagan: why do you say is not work, what is the exact issue you have? [15:19] heheh [15:19] mandel: its still opening the window [15:21] fagan: try with if blah is not None: [15:21] k [15:21] fagan: also, it would be a better things to do to disable the button when nothing is selected, and re-enable it when there is something, right? [15:22] mandel: that didnt work [15:22] ein? [15:22] id say it would be easier to do the button select thing [15:22] mandel: the window still opened [15:22] fagan: can you push the code,, because in my case, it work [15:22] s [15:22] weird [15:22] well, at least the version I wrote [15:22] mandel: windows or ubuntu? [15:23] windows, but I can try in any other os [15:23] should not matter [15:23] pushed [15:24] mandel: well the versions of Qt may be different [15:25] CardinalFang: So, although I merged greendroid to extract-login, and then extract-login to trunk -- launchpad won't include the "Merged branch" notice for the former? (I have foreseen this and made a commit msg like this: "Merge (+greendroid) extract-login to trunk.") [15:26] CardinalFang: I managed to untwist my stuff and hopefully correctly merged to trunk greendroid and extract-login. [15:26] fagan: yes, but that was worked at least since Qt3, I'm sure is something in the code, not in Qt [15:26] I'd say I'm 200% sure [15:27] mandel: well give my one a try and if it doesnt show im right :) [15:27] karni, I don't know what launchpad will do. I suspect it marked as merged every proposal that are subsets of whatever is landing in the branch. [15:27] CardinalFang: wow wait.. it says it merged greendroid branch o_O [15:27] CardinalFang: Okey, I need few more minutes on this heh ;) [15:32] ralsina_: ping [15:33] mandel: pon === ralsina_ is now known as ralsina [15:33] ralsina: do you have a windows vm I can borrow some time from? [15:34] ralsina: I''m got a small issue with the ubuntu-sso-client.exe and would like to test it in a diff system [15:34] mandel: no, but I have a real windows machine :-) [15:34] ralsina: real is also good :) [15:34] ralsina: I'll send you the mail [15:34] wtf I've merged extract-login (forked from greendroid) and it says it merged greendroid, although it has changes from extract-login as well >_< [15:34] bzr is becoming like a woman to me, will I be able to understand it? /me traces back his steps [15:35] mandel: done [15:35] I did the diable bit [15:35] and enable when they click [15:36] fagan: but what was the issue with the other code? [15:36] I'd like to know why it was not working [15:36] karni, so, extract-login is a superset of greendroid. You can't land or merge extract-login without also getting greendroid. [15:36] CardinalFang: hah.. I went to extract-login directory, and merged the greendroid branch there. Then, I went to trunk and merged extract-login. Thus, it says I have merged greendroid (which now contained changes from extract-login) [15:36] mandel: dont have a clue [15:36] CardinalFang: ↑ does it make sense what I wrote? [15:36] karni, ^ [15:37] mandel: its not something id dwell on though [15:37] the other way was a better way of doing it [15:37] CardinalFang: right, but I thought it'd display 'merged extract-login', *but* since I merged in the greendroid just before I merged extract-login to trunk [15:37] fagan: I'm kinda interested… 'cause I suspect that you are disabling the button in a similar fashion, and it works, right? [15:37] CardinalFang: extract-login now became greendroid, and that's why lp says greendroid has been merged. [15:37] karni, it should say both are merged. [15:38] * karni shits his pants [15:38] I'm scared ;d [15:38] mandel: I disabled it in qt designer and listened for the selection changed signal and enabled it then [15:38] karni, I think it should. [15:38] so they have to click [15:38] CardinalFang: I think it should, indeed.. [15:39] fagan: well, if you dont have the code, I'll guess thare was something wrong around it [15:39] mandel: so its not the same way of doing it [15:39] karni, if merge-login is greendroid + something, then merging merge-login will also bring greendroid. [15:39] mandel: well I gave you the entire bit of code [15:39] that was the pasted bit [15:39] CardinalFang: that makes perfect sense. but where's the "something" has been merged as well notice :< [15:39] and then I tried it with is not and it still didnt work [15:39] it was very interesting [15:40] CardinalFang: I'll poke around for a little longer.. [15:40] fagan: ok then, push it and I'll take a look [15:40] mandel: already done [15:41] ok, give me some mins, I have to restart the windows vm [15:41] * fagan makes tea [15:41] CardinalFang: bzr info in my u1f-extract-login branch says at the bottom: submit branch: /home/mike/src/android/ubuntuone/u1f-gd -- what does it mean? [15:42] CardinalFang: man I'm bothering you.. you're on the sprint, aren't you [15:43] karni, if you push to the destination, then every proposal that is a strict subset of all the changes that are landing will be marked as Merged. [15:43] greendroid has, but the 'top lever' extract-login hasn't [15:44] and that's what makes me wonder [15:45] karni, "cd local/extract-login; bzr missing lp:~karni/desktopcouch/extract-login; bzr missing lp:desktopcouch" [15:45] * fagan back [15:45] nessita: bug #702176 [15:45] Launchpad bug 702176 in ubuntuone-client (Ubuntu Natty) (and 3 other projects) "Syncdaemon needs to open the control-panel to volumes when a folder shared to the user exceeds the owning user's quota (affects: 1) (heat: 6)" [Medium,Invalid] https://launchpad.net/bugs/702176 [15:46] Ooooh I got a nice error on the cp on sat when I setup u1 on my new machine [15:46] CardinalFang: thanks! [15:47] mandel: got the email, will gert back to you in 1' [15:47] ralsina: superb, I dont know what the hell is py2exe doing and I keep getting a runtime error :( [15:47] karni, Don't mind asking me. It's important to get you mostly self-sufficient. [15:48] CardinalFang: Uhm. [15:48] Bug #748276 [15:48] Launchpad bug 748276 in desktopcouch (Ubuntu) "desktopcouch-service crashed with PreconditionFailed in request(): ('file_exists', 'The database could not be created, the file already exists.') (affects: 1) (heat: 8)" [Undecided,New] https://launchpad.net/bugs/748276 [15:48] fagan, I'm on that right now. [15:48] fagan: chad just fixed that [15:48] nice [15:49] * fagan loves that term "PreconditionFailed" sounds so dirty [15:50] fagan: so you got that bug? [15:50] thisfred: yep [15:51] fagan: ok, then we could really use your help testing the fix :) [15:51] mandel: is the sso.exe in the mail? Because it's taking forever to download from canonical's IMAP [15:51] sure [15:51] we've not been able to reproduce it here [15:51] fagan, ...assuming it happens somewhat often or reliably. [15:51] fagan: we'll ask you when there's something ready [15:52] CardinalFang: well I got it the first time around using u1 on my netbook [15:52] ralsina: yes, is quite big, 16,3 MB [15:52] so I dont know if it happens relibably [15:52] 16,3 MB isnt that big [15:52] fagan: I'd like it to be a lot smaller [15:53] mandel: well its smaller than firefox and chrome on windows [15:53] so its small enough [15:53] but we can compress stuff so it can go smaller [15:55] CardinalFang: I have overwritten the trunk back to revision 52 (I know, I won't do that again). Now merged greendroid branch, doing it safely one by one this time. [15:58] mandel: can you put it somewhere I can get it via HTTP? Something weird happens with that file over IMAP [15:58] * mandel wonders what is going on with his machine.... [15:58] fagan: the good side is that adding the rest of u1 there will MAYBE add .5MB extra [15:58] ralsina: sure, on it [15:58] ralsina: u1 should be the right place, right? [15:59] ralsina: well the installer wouldnt be too big anyway [15:59] mandel: +1 for marketing purposes ;-) [15:59] * fagan checks whats taking up all of his ram [16:00] (compiz is taking up 833 MB /me thinks memory leak) [16:01] Im going to restart brb [16:04] that was a lot of memory compiz was taking up [16:05] so now im at 22% instead of 60% thats not bad :) [16:06] CardinalFang: phew` \o/, merging branches to trunk one by one is a simple and good approach. merged greendroid, merged extract-login. [16:07] :) [16:09] hehe im breaking everything today [16:09] sso just broke on the u1 website for me [16:11] fagan: sorry I have not been helping out with the training today, but it has been a busy morning [16:11] fagan: I will be more available tomorrow [16:11] * karni reviews photo-upload-current [16:11] ralsina: its cool im still fiddling with the thingy from friday [16:12] and im having fun breaking stuff :D [16:12] fagan: ok, I guess ;) [16:12] fagan: I'll review yur code in a second, the machine has decided to go nuts... [16:13] * fagan just realised he shouldnt say he is having fun at work since fun kinda means slacking off === alecu__ is now known as alecu [16:14] im on till 5:30 mandel so take your time [16:14] ralsina: I'm going to reboot the bloody thing to see if it become usable, will be back asap [16:14] mandel: cool [16:15] ralsina: if you have time you can review it for mandel [16:15] fagan: show it to me [16:15] fagan: fun is good, no problem there [16:16] ralsina: lp:~shanepatrickfagan/+junk/string_list [16:16] fagan: ok, I'll take a quick look [16:16] it should be fine except a load of pylint warnings about features and and names of some vars [16:16] * fagan needed to break 1 or 2 rules to get it working right [16:17] fagan: ok, I'll check that [16:23] fagan: I am pretty sure "up" and "down" should move the current item around, but wait for mandel to confirm it ;-) [16:23] ralsina: yeah mandel and I talked about that earlier [16:23] its not biggy [16:24] *no :) [16:25] that was one of those interperatation things [16:26] ok [16:26] cool, so let's look at the code [16:26] * fagan should comment more [16:30] fagan: suppose I told you I would prefer the buttons to group at the top of the column instead of being evenly spaced. What would you do? [16:31] ralsina: well I could stick in a vert spacer that expands [16:31] ok, right answer ;) [16:31] yay [16:35] tally me banana [16:35] dobey: what does that mean :) [16:42] ralsina: so is the code up to scratch? [16:42] other than the up down thing [16:43] * fagan is still programming like a C developer at times :) [16:44] ralsina, thisfred, alecu: were any of you aware of the removal of the emblem api in unity? [16:44] joshuahoover: yeah, we all were [16:44] thisfred: ok, cool...just wanted to make sure since it's kind of late in the release :) [16:44] it's unfortunate, but we didn't use it for anything that is critical to communicate to the user\ [16:46] dobey: can you please share with me the verify_command line for u1client for tarmac? [16:47] maybe [16:53] CardinalFang: "We might try to upload this. This makes Chad cry. XXX" hahahha [16:57] karni, ah, for zero modtime. Yes. [17:04] ralsina, fagan: I'm back, my machine was going nuts, had to get the backup hd out... [17:05] /join #apple [17:06] mandel: its good now though [17:06] ? [17:08] fagan: yes, I take hourly backups, so not big deal, I just lost some time [17:08] hourly backups thats interesting [17:08] * fagan just puts stuff on u1 [17:09] ralsina: here is the file: http://ubuntuone.com/p/ksa/ [17:09] mandel: its good now though lp) [17:09] damn it [17:09] fagan: well, I dont have enough space to back up the 3 windows vms I have, so the backup approach scales better :) [17:09] mandel: oh so you test vista, 7 and xp? [17:10] yes [17:10] Shame you dont do windows ME [17:10] so many people use it still and we could corner the market on all 5 of them [17:11] I think we are dong enough with vista already, that guy is a pain ;) [17:11] mandel: so you have 15 minutes to check out my branch [17:12] 15~ [17:12] fagan: on it, did ralsina take a look? [17:13] mandel: yeah he did [17:13] he asked me a question too which I got right :) [17:13] He just commented on the up down thingy [17:13] that was it [17:16] fagan: the up and down is about why it does not move items up and down, right? [17:17] mandel: well that was me thinking it was for nav rather than organisation [17:17] but yeah that [17:20] fagan: fancy trying to implement that?I mean, organizing the items [17:20] should be a few lines more than you have [17:20] mandel: sure [17:20] ok, go for it then :) [17:21] mandel: is that the only thing im missing? [17:21] CardinalFang: huh, I left you quite a comment lol ;D reviewed! [17:22] Do we know why the emblem API was removed? [17:23] fagan: From the requirements, I think it is, Ill take a look and review the code as soon as you have that last thing done [17:24] mandel: oki doke almost done [17:27] mandel: pushed [17:27] and teste [17:27] and tested [17:29] ok, on it [17:31] I should do some error handling for if they have no rows selected or no rows [17:31] * fagan does that [17:32] fagan: since you are touching the code, also it would be nice if you have selected and item and moved it up or down, that the current item is the one you moved [17:32] * mandel hopes he makes sense [17:33] fagan: mainly so that if you do to ups, you move the item twice up, rather than move, clicked moved item, move [17:33] ok thats not hard [17:42] * mandel hates py2exe... [17:43] mandel: fixed and pushed [17:43] and tested [17:43] and im done for the day [17:43] but ill hang around to see the review [17:44] * mandel looks [17:48] fagan: I think that you should have added to the buttons to be disabled the remove one, right? I mean all the others are and removing does follow the same logic the only thing is that ti does not crash [17:48] mainly because remove can remove something that is not there... [17:48] mandel: oh forgot about that one [17:49] * fagan adds [17:49] anything else [17:51] fagan: in the title of the QInutDialogs: is better to do 'Add %s' % self.list_type [17:51] right? [17:51] yeah [17:51] I did it with + though [17:51] but its the same really [17:51] fagan: there is also no need to use \ to brake a statement when that statement is within brakets [17:52] mandel: I did that because the lines were too long [17:52] I could have shortened them by putting them into vars though so i could have fixed it pretty easy [17:53] fagan: yes, but is nicer for translations to have the full string [17:53] asking thranslators to work around broken sentences does not lead to good places :) [17:53] yeah point taken ill do that from now on [17:54] so is that it? [17:54] fagan: you can still use the + right: ('blah blah blha' + 'blah %s') % thing [17:55] fagan: I'm still looking [17:55] I dont mind using the %s [17:57] fagan: since you are not passing anything apart from the default parameter to sortItems, you could have connected the sortItems slot with the clicked signal [17:57] that way you would not need the extra method [17:59] mandel: well I could but I like to make the code a little bit easier to read so you can see oh its doing something here rathar than I dont have a clue whats happening here [17:59] it just seems easier to read [17:59] for me anyway [18:00] fagan: there is no need to loop around the list of strings, there is an addItems method [18:00] mandel: oh didnt see that there [18:00] thats easily fixed [18:00] fagan: well, yes but you have to get used to the way and advance developer would do it, and hew would probably would not add an extra method [18:01] fagan: I once had a friend at uni that did not like arrays and she preferred to declare all vars, it was easier for her, but we both can agree that does not scale very well :) [18:01] so, lets try to make the best code possible [18:02] mandel: hah im not that bad :) [18:02] * fagan prefers pointers to arrays anyway [18:04] ok im off anyway I pushed that last change into the branch [18:04] other than the method thing which I just prefer myself but I can fix it pretty easily [18:05] fagan: one last one, can you pass the list_type to be an instance variable that is set through the constructor? [18:06] fagan: I would not be surprise if in a normal review they will ask you to connect the things directly, the less code one writes, the less number of bugs :) [18:06] fagan: I'm off too, I need to walk the dog, catch you tom [18:06] kk [18:07] later mandel === zyga is now known as zyga-dinner [18:30] CardinalFang: did you ever land https://code.launchpad.net/~cmiller/ubuntuone-android-music/play-from-file-descriptor-not-path/+merge/43839 ? [18:32] dobey, it's in the current release candidate that we're trying. It appears to fail for MP4 files, so I poked a hole for that condition. [18:32] can I have some reviews on this? https://code.launchpad.net/~alecu/ubuntuone-client/fix-nautilus-symlinks-udfs/+merge/56225 [18:34] CardinalFang: ah, it is "Approved" so i was wondering. should it probably be changed to merged? === zyga-dinner is now known as zyga === tubadaz___ is now known as tubadaz [19:28] one more review, please? https://code.launchpad.net/~alecu/ubuntuone-client/fix-nautilus-symlinks-udfs/+merge/56225 [19:30] can I have a couple of reviews for: [19:30] https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/disable-files/+merge/56217 and [19:30] https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-to-translate/+merge/56242 ? [19:30] alecu: I'll review yours [19:31] * nessita brbs [19:53] dobey: i forgot to mention that bug #733327 is now approved [19:53] Launchpad bug 733327 in libubuntuone (Ubuntu) (and 1 other project) "[UI FFE] Notify user of missing MP3 support (affects: 1) (heat: 151)" [High,New] https://launchpad.net/bugs/733327 [19:54] joshuahoover: i saw. thanks! [19:56] nessita, comments on how to test added to the nautilus branch [19:56] alecu: thanks [19:58] thisfred, https://code.edge.launchpad.net/~cmiller/desktopcouch/preconditionfailed-lp707321 [19:59] Greetings. I showed up here a few days ago because ubuntu one had replaced all of my contacts on my phone with blank contact information. Upon checking the status, there were some problems, but the reported problems do not match my issue. I'm unable to view my contacts on one.ubuntu.com and I am concerned I have just lost 241 contacts. Where should I go from here? [20:01] fishscene: You should stick around, this is the place where you should get some answers (not directly from me, though). [20:01] fishscene: hmm, if this is related wth the mobile plan I thin beuno is the right person to talk to [20:02] ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/disable-files/+merge/56217 and https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-to-translate/+merge/56242 [20:03] fishscene: actually I think joshuahoover is the right person to start talking to, I'm sure he can help you or know who can help you [20:03] joshuahoover: ping [20:03] Yes, I'm using the ubuntu one mobile client on my Nexus S. I'll stick around for as long as I can, but I'm at work and might be called away suddenly. I really would appreciate any assistance. :) [20:03] mandel: pong [20:03] Should I PM joshuahoover? [20:03] ^ and there he be [20:04] fishscene: have you reported a bug about it? [20:05] fishscene: with a bug number we can start doing some work even if you are not around, you can even add private data and set the bug to be private so that only the u1 developers can see the info [20:05] * mandel is done for the day... [20:05] mandel: Nope. I have been unable to find the bug reporting area. Truth be told, I haven't scoured the entire ubuntu one website either. [20:06] joshuahoover: can you give a hand to fishscene he seems to have a fairly big issue with his contacts on his phone [20:06] I have no clue who should be handling his problem [20:06] I'm PM'ng him :) [20:06] fishscene: unless there is private info, around here is ok [20:06] ok [20:11] mandel: now I recall, beuno has mentioned fishscene (or similar person) with gone 200+ contacts. too bad he isn't around ATM.. [20:13] ah. yea. After I posted in here on Friday(?), I was called away and was unable to pursue any support. :( [20:19] oh, just out of curiosity, is there an estimated time when contacts will be viewable on one.ubuntu.com? === ivanka is now known as ivanka-train [20:33] fishscene, hi [20:33] * rye is reading the backlog [20:33] ralsina, por favor. https://code.edge.launchpad.net/~cmiller/desktopcouch/preconditionfailed-lp707321/+merge/56246 [20:34] fishscene, do you have the access to an ubuntu-based machine? [20:34] Hi rye [20:34] rye, possibly [20:34] Depends on what needs to be done on the ubuntu machine [20:35] fishscene, when you go to https://one.ubuntu.com/contacts/ do you have the records that "Contact Name was removed" ? [20:36] I get a webpage: Something has gone wrong Sorry about that. This problem has been reported to Ubuntu One engineers and we'll fix it as quickly as possible. [20:39] CardinalFang: por supuesto! [20:39] Some contacts may have been deleted, but that was *right* before I realized what had happened. [20:39] Maybe about.. 5 contacts or so [20:40] ralsina, alecu_: https://bugs.edge.launchpad.net/ubuntuone-control-panel/+bug/747677 [20:40] Launchpad bug 747677 in unity (Ubuntu) (and 9 other projects) "[FFE] Need API to set urgency from background process (affects: 1) (heat: 12)" [High,Confirmed] [20:40] fishscene, ok. First of all you should be able to get the contacts as a big file using http://people.canonical.com/~roman.yepishev/us/ubuntuone-couchdb-query from the Ubuntu machine and account which has Ubuntu One pairing [20:40] https://code.edge.launchpad.net/~thisfred/ubuntuone-control-panel/remove-urgency-on-focus/+merge/56250 [20:42] just a moment.. [20:43] fishscene, to get the file you need to run the script I gave the link to as python ubuntuone-couchdb-query contacts/_all_docs?include_docs=true > contacts.json [20:43] ok just a sec. [20:44] fishscene, another question - If you open a source view of the web page with "Something has gone wrong" you should be able to locate OOPS word and an identifier of one, It may be helpful to diagnose what has happened. [20:44] fishscene, additionally I have a script to recover all the deleted documents from ubuntuone couchdb which you might want to use, but I'd like to get the backup first [20:45] OOPS-ID-1920appserverZHGIdBdEfJDAAEDDabDCAffIHFdCIaBAb146217 [20:45] https://lp-oops.canonical.com/oops.py/?oopsid=ID [20:46] beuno, sso-test-8@rtg.in.ua on a second couchdb shard seem to be having problems [20:46] fishscene, do you have "SOmething has gone wrong" on https://one.ubuntu.com/auth/login/ ? [20:47] nope. Everything works 100% fine unless I click on the contacts link [20:47] beuno, nm, i had no subscription, but it looks we are not handling that well [20:50] whoa [20:50] ok. What do I do with this contacts.json file? === 15SAAFJHR is now known as tubadaz [20:52] fishscene, what size is it ? [20:53] on Windows, 325KB, on Linux, 325.7KB [20:53] Linux exact: 333514 [20:53] bytes [20:53] fishscene, in case everything else fails I'll write a converter from the json format from couchdb to something parseable [20:53] fishscene, ok, now let me test the script I have... [20:55] fishscene, ok, now please download the script from http://people.canonical.com/~roman.yepishev/us/ubuntuone-couchdb-undelete.py and run it as python ubuntuone-couchdb-undelete.py --dry-run contacts [20:56] ok done [20:57] I saw a bunch of contacts fly by... at the end, the output was: "Adding the document to undelete queue" "Found 62 deleted documents" "Not doing anything since --dry-run is specified. [21:01] mandel, ping [21:01] fishscene, 62 deleted documents... not the number you are expecting to be [21:02] True. one.ubuntu.com says I have 231 contacts [21:03] So that's the number I've been using. I have been unable to verify if I actually have 231 contacts :P (This number was 241 a few days ago?) [21:07] I have to run to a meeting. Sorry. I shall return as soon as possible. [21:08] fishscene, wait, i think i misunderstood the request, you _have_ 231 contacts listed, but you cannot access them due to /contacts/ failure, right? [21:08] fishscene, ok, i think I'll be here [21:09] thisfred: can I have one more review for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-to-translate/+merge/56242 and for https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/disable-files/+merge/56217 ? [21:33] CardinalFang: I hope my review will prove useful. I didn't mean to be picky, you know that. More curious :) === fishscene is now known as fishscene_afk [22:45] can I have a couple of reviews for https://code.launchpad.net/~nataliabidart/ubuntuone-client/better-path-handling/+merge/56262 ? [22:58] ralsina: https://code.launchpad.net/~nataliabidart/ubuntuone-control-panel/better-path-handling/+merge/56264 [23:07] ralsina: bug #748585 [23:07] Launchpad bug 748585 in ubuntuone-client (Ubuntu) (and 1 other project) "ubuntu one control panel don't find the informations (affects: 1) (heat: 6)" [Undecided,Invalid] https://launchpad.net/bugs/748585 [23:43] Happy birthday popey, if you happen to still be arounf. [23:56] ok! I'm back