| dpm | good morning all | 07:34 |
|---|---|---|
| happyaron | morning dpm, :) | 07:34 |
| dpm | good afternoon happyaron :) | 07:34 |
| andrejz | good morning dpm, afternoon happyaron :) | 07:39 |
| dpm | hey andrejz ;) | 07:40 |
| happyaron | andrejz: :) | 07:41 |
| andrejz | got some good news. number of visits on slovenian ubuntu mirror has increased by 40% in May 2011 compared to May 2010 :) | 07:42 |
| TLE | dpm: hey | 10:27 |
| TLE | dpm: I have been looking a little around the wiki, where was it that you wanted us to document the tasks under sustainable translations? | 10:28 |
| TLE | put the under Actions here https://wiki.ubuntu.com/UbuntuTranslationsCoordinators ? | 10:28 |
| TLE | or is there some dedicated space? | 10:28 |
| RawChid | Hello | 11:08 |
| dpm | TLE, sorry for the delay, I haven't done it yet, let me see if I manage today. | 11:08 |
| dpm | hey RawChid :) | 11:11 |
| RawChid | Hola dpm, do you have time to discuss my code? | 11:12 |
| dpm | RawChid, sure. Could you give me the link to your branch again? | 11:13 |
| TLE | dpm: no problem, the only reason I ask is that I keep asking myself (and then eventually you) the same questions, might make sense to start documenting | 11:14 |
| RawChid | dpm: https://code.launchpad.net/~rachidbm/ubuntu-l10n-tools/pot-priority | 11:15 |
| RawChid | We discussed to have a central place for the list of RELEASE_NAMES, now I'v put it in ul10n_toolsconfig, I doubt that's a good place, but didn't know a better one... | 11:17 |
| RawChid | I couldn't find how to get a list of names from Launchpad. Second, if retrieved from launchpad, this will slow down the script unnecessary IMHO | 11:18 |
| dpm | TLE, if you've got any question in mind already, feel free to post it here and it will be saved in the log. I'll then make sure it gets documented | 11:19 |
| dpm | RawChid, I think it might still be better to get the releases from LP, as the script won't be slowed down too much (we're already connecting to Launchpad even if we're not fetching the release names). If you want to postpone that for now, it's fine, but I'd like to ask you to change one thing: right now whenever a new release is opened we'll need to change the code (whereas reading from the LP API would spare us that step). While this is ok'ish fo | 11:23 |
| dpm | r now, the code should probably only be changed in one place. Now it needs to be changed in two places (adding a new release to the list, and setting the default release) | 11:23 |
| RawChid | Hm, okay. And what is the default release at the moment? | 11:25 |
| dpm | I meant the default release you set on the script (I think you set it to natty). | 11:26 |
| RawChid | Yes, that is what I've done. But I want to know which YOU want/need to be default | 11:27 |
| RawChid | So Natty is good? | 11:27 |
| dpm | RawChid, I think one really nice thing would be to detect the release the user is running, and make that the default | 11:28 |
| dpm | RawChid, I think you should be able to get the Ubuntu release names with the series_collection property from https://launchpad.net/+apidoc/devel.html#distribution | 11:28 |
| RawChid | I've investigated that part of the API, but didn't come far. | 11:28 |
| RawChid | Oke, default is the one that is on users system, sounds good to me | 11:29 |
| RawChid | So it would be an idea to make a function in launchpadmanager.py get_distro_codenames() ? | 11:32 |
| dpm | RawChid, here's how you can do it: | 11:34 |
| dpm | http://pastebin.ubuntu.com/622465/ | 11:34 |
| RawChid | OMG, it could be so simple | 11:34 |
| RawChid | I was think way to difficult :S | 11:35 |
| RawChid | thinking* | 11:35 |
| dpm | yeah, the LP API is very nice when it provides the thing you need. If what you need is not exposed in the API, then you're generally screwed :) | 11:37 |
| dpm | RawChid, and here's how you can detect the distro the user is running: | 11:37 |
| dpm | http://pastebin.ubuntu.com/622466/ | 11:37 |
| RawChid | Great, I'll change that. And will take over your implementation of the CSV reader | 11:38 |
| RawChid | Do you agree with function in launchpadmanager.py get_distro_codenames() ? | 11:40 |
| dpm | RawChid, yeah, I think it's exactly the place where it should be | 11:41 |
| dpm | when you're implementing that function, you might want to discard all the '*-series' series from the list it returns, since these are only series that have been registered, but not yet open (i.e. there are no translation templates we can modify there) | 11:43 |
| RawChid | Ack | 11:45 |
| RawChid | And things as warty and hoary are still interesting? :P | 11:46 |
| dpm | wait, there's probably a property that tells if a series is active or not, let me read the documentation | 11:49 |
| dpm | RawChid, yeah, there is an 'active' property on https://launchpad.net/+apidoc/devel.html#distro_series, so you should only return those series where series.active == True | 11:50 |
| RawChid | Oke, great | 11:52 |
| RawChid | Hm, this is more a Python question. I can loop al the series, and add each (active == True) to a list. But I guess there is a more clever way to do this in Python | 12:14 |
| RawChid | dpm, if you know more reading material on this stuff :) | 12:16 |
| dpm | RawChid, that's also how I'd do it (initialize an empty list, iterate over series, append the active series names to a list), I don't know if there is a cleverer way, you might want to ask on #python | 12:19 |
| RawChid | ack | 12:20 |
| TLE | if you are talking about reducing a list based on a parameter in the elements in the list, you can do that with a list comrephension with an if in | 12:21 |
| TLE | ? | 12:22 |
| RawChid | Yes, I think something like that | 12:23 |
| TLE | >>> h=['active','active','inactive','active'] | 12:23 |
| TLE | >>> g=[a for a in h if a=='active'] | 12:23 |
| TLE | >>> g | 12:23 |
| TLE | ['active', 'active', 'active'] | 12:23 |
| TLE | I even _think_ that is the "Pythonic" way of doing it ;) | 12:24 |
| RawChid | I agree, but I need to do some more reading to invent such a solution by myself :P. Thanks TLE! | 12:25 |
| TLE | np, well, plenty of good reading around | 12:26 |
| RawChid | I'm open for suggestions ;) | 12:27 |
| TLE | I just finished Dive into python, that's nice, but also book length | 12:27 |
| RawChid | Okay | 12:27 |
| TLE | If you are looking for something shorter, I sure there are some nice tutorials around, but I don't know any of the top of my head | 12:28 |
| dpm | yeah, dive into python is really nice and well written. I haven't read it all myself, rather some chapters for reference when I needed to find out about something in particular | 12:28 |
| TLE | But actually I can really recommend Dive into Python, because you just "dive in" right away, and he really docusses on teahcing you the elegant way of doing stuff | 12:29 |
| TLE | http://diveintopython3.org/ | 12:29 |
| TLE | if you want to do python 3 else http://diveintopython.org/ | 12:30 |
| RawChid | Hehe, already downloaded :) | 12:30 |
| TLE | The only real bummer is that I bought a dead tree version, because I wanted to support the Author and all, and I ended up with one (part of a bunch of book printed that way) with a printing error in | 12:31 |
| TLE | so there was no horizontal lines in all the code examples | 12:31 |
| TLE | do you have any idea how difficult it is to read code without - and _ ;) | 12:31 |
| TLE | But that is the fault of the printing company not the author | 12:32 |
| TLE | in any case if you get the same idea as me about supporting the Author I would recommend just using the online version and sending him a donation ,) | 12:32 |
| RawChid | :-) | 12:34 |
| dpm | RawChid, here are the two approaches: http://pastebin.ubuntu.com/622503/ | 12:35 |
| RawChid | Hm, now I know why I wanted to prevent getting them from launchpad. I printed the optional series in help. Should I just remove the series from help text? | 13:05 |
| TLE | dpm: hey | 13:17 |
| TLE | about the Maverick language packs, did you say that the last ones that were build were diffs, so that we could not use those for testing, or was it merely that they could not be used for the FF update stuff | 13:18 |
| dpm | TLE, sorry, I should have been more clear. I meant the second: i.e. they could be used for testing, but not this particular time due to the FF changes | 13:20 |
| dpm | for the FF changes we need a full export | 13:20 |
| TLE | yes ok, so we wait for response on the email you just wrote, about whether ff5 i actually comming to maverick, and if not, we are good to go, otherwise we wait, right? | 13:23 |
| TLE | dpm: ^^ | 13:30 |
| dpm | TLE, sorry, I was away for some minutes. Ack on your last sentence | 13:31 |
| RawChid | dpm! Don't know if you saw my MP already, but I forgot something, so I do a resubmit right now | 14:33 |
| dpm | RawChid, sure, no worries | 14:37 |
| dpm | RawChid, have you actually tested it before submitting it? I see a couple of things that probably stop it from it working properly: | 14:40 |
| dpm | http://pastebin.ubuntu.com/622582/ | 14:40 |
| RawChid | Yes I tested it. I didn't knew the "header check" isn't necessary anymore | 14:42 |
| RawChid | NO offense, do you test your code before submitting? Because you're CSV reader only save the last entry ;) | 14:43 |
| RawChid | forgot an ident ;) | 14:43 |
| RawChid | I'll remove that header thing | 14:44 |
| RawChid | Hm, it contains more errors, I did the resubmit a bit hasty :'( | 14:47 |
| dpm | RawChid, good point ;) I didn't test my last commit, no, I just tested the DictReader stuff in a console and just made a checkpoint commit to have the csv stuff in - I now realize that the missing indentation makes it only save the last template. But on the other hand, whenever I submit a merge proposal to someone else's branch I do always test it, so that the reviewer doesn't have to play catch with bugs introduced by the branch. I'm not complain | 15:01 |
| dpm | ing, though, you say you did test it, so it's fine :-) | 15:01 |
| TLE | dpm: jost wrote to the list about a small post ponement, so now we can just start the cycles when the packs are ready | 15:11 |
| dpm | TLE, which list? | 15:14 |
| TLE | ubuntu-translators | 15:15 |
| TLE | just to let people know why we haven't announced testing starts, when it's on the schedule | 15:16 |
| dpm | ah, I don't seem to have received the message, let me refresh again... | 15:16 |
| serfus | i did :) | 15:17 |
| dpm | ok, it was just my Evolution being slow :) | 15:17 |
| dpm | TLE, read the message, ack and thanks ;) | 15:17 |
| TLE | no need to thank me for that, that is the task I agreed to ;) | 15:19 |
| serfus | something is funky with synaptic, but i have to leave now.... i will probebly nag you guys with this later | 15:19 |
| TLE | just entered the 10's: Made a twitter account, found a few feeds to follow and hooked it into gwibber, I feel so connected | 15:20 |
| dpm | hehe | 15:24 |
| dpm | anyway, I need to go earlier today, so I hope everyone has a nice rest of the day | 15:25 |
| TLE | you too | 15:25 |
| dpm | RawChid, if you've got more changes, do submit them, but I might not be able to look at them until tomorrow. Keep up the good work! | 15:25 |
| RawChid | No problem, thanks in advance. I'm not going to work on this for the next days | 15:34 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!