/srv/irclogs.ubuntu.com/2010/04/22/#launchpad-reviews.txt

=== EdwinGrubbs changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [adiroiban(UI bug-146178), adiroiban(bug-61081)] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
mwhudsonrockstar: https://code.edge.launchpad.net/~mwhudson/launchpad/fix-ec2-harder-dammit/+merge/2389704:03
=== adiroiban changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [adiroiban(UI bug-146178), adiroiban(bug-61081), adiroiban(bug-487137)] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
thumperrockstar: https://code.edge.launchpad.net/~thumper/launchpad/yet-more-api-exports/+merge/2390505:43
=== henninge_ is now known as henninge
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775 || reviewing: adiroiban(UI bug-146178) || queue: [adiroiban(bug-61081), adiroiban(bug-487137)] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== ctrlsoft changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: adiroiban(UI bug-146178) || queue: [adiroiban(bug-61081), adiroiban(bug-487137)] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
noodles775Hi ctrlsoft :)09:25
=== ctrlsoft is now known as jelmer
jelmermoin noodles :-)09:25
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: adiroiban(UI bug-146178) || queue: [adiroiban(bug-61081), adiroiban(bug-487137), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
noodles775intellectronica: Hi! are you after more UI reviews? If not, I'll do it now, just let me know what you prefer (for https://code.edge.launchpad.net/~adiroiban/launchpad/bug-146178/+merge/23760)09:29
noodles775wgrant: hi! do you want me to land this one as is? https://code.edge.launchpad.net/~wgrant/launchpad/emailauthentication.txt-2.6-fix/+merge/2344909:29
wgrantnoodles775: Yeah, I'm a bit busy until Wednesday, so it'd probably be a good idea.09:36
wgrantI'll track down the real bug then.09:36
noodles775wgrant: ok, will do.09:37
wgrantThanks.09:37
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: adiroiban(bug-61081) || queue: [adiroiban(UI bug-146178), adiroiban(bug-487137), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
henningenoodles775, jelmer: I have a CP-candidate that could use some high-priority treatment. Can either of you take that?10:04
jelmerhenninge: I'll have a look10:07
jelmerhenninge: where's the MP?10:07
henningehttps://code.edge.launchpad.net/~henninge/launchpad/bug-565294-nplurals/+merge/2386110:07
henningejelmer: thanks10:07
henningejelmer: it looks large but a lot of is was me removing lint.10:07
henningeI probably should not have done that in this branch ...10:08
jelmerhenninge: it looks like the lint changes are mostly removing assignments?10:09
henningejelmer: yes, unused variables10:09
jelmerhenninge: is it correct that test_header_pluralform_western uses 'ja'?10:31
jelmer(which is Japanese I guess?)10:31
henningejelmer: yes, it is about the case were the western standard formula is found in the file for a language were that should not be.10:32
henningeand a language that has less plural forms than 2 - which is 1 ...10:32
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: adiroiban(bug-487137),henninge || queue: [adiroiban(UI bug-146178), adiroiban(bug-487137), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: adiroiban(bug-487137),henninge || queue: [adiroiban(UI bug-146178), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
jelmernoodles775: thanks10:33
noodles775np, just updating what I'm reviewing currently :)10:33
henningejelmer: the actual logic behind that is based on Danilo and me looking at the current data in the database.10:34
henningejelmer: and we found some po files for asian languges that report western plural expressions.10:34
henningein many templates for POFile headers, the western style is used as the default and often translators just don't change that before importing the file to Launchpad.10:35
jelmerhenninge: ah, ok - thanks10:35
StevenKadeuring: Hi! Do you have a moment to look at https://code.edge.launchpad.net/~stevenk/launchpad/fixes-bug-451396/+merge/21706 again?12:20
adeuringStevenK: sure12:20
StevenKadeuring: Ignore the 'import urllib', please, I'm just pushing the branch that gets rid of that12:20
adeuringStevenK: ok ;)12:20
intellectronicanoodles775, jelmer: can one of you review a small branch for me?12:38
noodles775Suer!12:39
jelmerintellectronica: Sure, if you give me 15m :-)12:39
intellectronicajelmer: great, thanks. just creating an MP.12:39
intellectronicajelmer: https://code.edge.launchpad.net/~intellectronica/launchpad/bug-heat-days-active/+merge/2392012:54
jelmerintellectronica: k12:54
intellectronicathankyou12:54
noodles775intellectronica: not sure if you saw my message before, but let me know if you are after UI reviews, otherwise I'll just do Adi's UI review now.12:57
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: -,henninge || queue: [adiroiban(UI bug-146178), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
intellectronicanoodles775: i must have missed your message. i'm happy to do a ui review. which mp?12:57
noodles775intellectronica: I just wasn't sure if you were trying to collect UI reviews atm. (as a mentat), otherwise I'm happy to do it... it's this one: https://code.edge.launchpad.net/~adiroiban/launchpad/bug-146178/+merge/2376012:59
intellectronicanoodles775: do you know if adi will be back today for an interactive review, or shall i just reply in the mp?13:03
noodles775intellectronica: I haven't heard back from him today after doing two code reviews, I'd do it via the mp for now.13:04
intellectronicaoright13:05
noodles775jelmer: have you started intellectronica's? If not, can I swap you and you can review mine? :)13:05
jelmernoodles775: I haven't started on his yet13:06
jelmernoodles775: So swapping would be fine :-)13:06
noodles775Great, although I guess I'll need to ask someone else to do the second review on mine :)13:06
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: intellectronica,henninge || queue: [adiroiban(UI bug-146178), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
StevenKadeuring: No news is good news? :-P13:08
adeuringStevenK: nah, things look fine :)13:09
intellectronicajelmer, noodles775: whoever is reviewing, please wait a sec, i just realised the diff is too much, because it contains some changes from a previous branch i thought had already been merged.13:09
noodles775intellectronica: ok, np.13:10
adeuringStevenK: I have just one remaining nitpick: could you remove the trailing space in the line "Leaving the PGP signature on a package uploaded " ?13:10
adeuringwe try to avoid them so that we don't get sprurious diff lines when somebody adds/removes trailing spaces.13:10
intellectronicacan i add a prequisite branch to the mp after the fact, or do i have to recreate it?13:10
StevenKadeuring: Okay, done.13:11
noodles775intellectronica: I can't see any way to do so without recreating either.13:11
=== bigjools changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: intellectronica,henninge || queue: [adiroiban(UI bug-146178), noodles, bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
bigjoolsnice branch for you guys :)13:11
adeuringStevenK: cool. r=me. I assume I should run your branch  through ec2, or can you do that?13:12
jelmernoodles775: so I guess I should get somebody else to review my review of your code?13:12
StevenKadeuring: I can do that. Can you fiddle the MP?13:12
noodles775jelmer: yeah, shouldn't be a problem though. But maybe grab bigjools' for the moment, as at least we can do that one together?13:13
adeuringStevenK: done13:13
intellectronicanoodles775: oright, the new mp is https://code.edge.launchpad.net/~intellectronica/launchpad/bug-heat-days-active/+merge/23921 .   sorry for the inconvenience.13:14
noodles775np.13:14
StevenKadeuring: Thanks! I'm throwing it through EC2 now.13:15
noodles775intellectronica: I can't see that days_since_last_activity is being used for anything?13:17
noodles775Did you mean to do (max_heat * 0.25 / (days_since_created - days_since_last_activity)) or something similar, looking at the comment?13:18
noodles775Ah, or looking at the related bug title, I'm guessing it should be s/dasy_since_created/days_since_last_activity on line 21 on the MP diff?13:19
henningejelmer: thanks for the review, I agree with you nitpicks ;)13:20
jelmerhenninge: sorry it took so long, I haven't really looked at the translations code before :-)13:20
henningejelmer: that's ok. Ask about whatever is unclear.13:21
henningeactually, I will go to lunch now ...13:21
henninge;)13:21
intellectronicanoodles775: no, i think that's a cut-n-paste error.13:21
henningejelmer: will you pass it on to noodles775 for mentoring?13:21
jelmerhenninge: Yeah, I've just requested a review from him as well.13:21
henningejelmer: cool, thanks13:22
daniloshenninge, jelmer: it's very important to get this CPed before 2200 UTC today13:22
* noodles775 switches to it.13:22
danilosnoodles775, thanks13:23
noodles775intellectronica: so should it be divided by the number of days *since* the bugs last activity (as stated in the bug title), or divided by the difference between the days since the bug was created and it's last activity (as the comment seems to suggest?)13:27
=== noodles775 changed the topic of #launchpad-reviews to: On call: noodles775, jelmer || reviewing: henninge,henninge || queue: [adiroiban(UI bug-146178), noodles, bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
intellectronicanoodles775: it should be divided by the days since the bug's creation13:28
noodles775intellectronica: so the bug 567439 title is wrong then, ok.13:28
mupBug #567439: Add MAX_HEAT / 4 / days since last activity to bug heat <story-bug-heat> <Launchpad Bugs:In Progress by intellectronica> <https://launchpad.net/bugs/567439>13:28
intellectronicanoodles775: oh, right, it is. there's another bug for a calculation based on time since last activity, i must have confused them.13:29
noodles775henninge: why not s/language_forms/number_plural_forms on line 57 and do away with the extra variable? (line 73 sets number_plural_forms to the same value?)14:00
noodles775Ah, right, as the language.plural forms can be overwritten with None... nm.14:01
=== noodles775 changed the topic of #launchpad-reviews to: On call: jelmer || reviewing: - || queue: [adiroiban(UI bug-146178), noodles, bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== matsubara is now known as matsubara-lunch
=== salgado is now known as salgado-lunch
=== deryck is now known as deryck[lunch]
=== matsubara-lunch is now known as matsubara
=== gary_poster is now known as gary-lunch
=== leonardr is now known as leonardr-afk
=== salgado-lunch is now known as salgado
=== leonardr-afk is now known as leonardr
=== deryck[lunch] is now known as deryck
=== gary-lunch is now known as gary_poster
=== sinzui changed the topic of #launchpad-reviews to: On call: jelmer || reviewing: - || queue: [adiroiban(UI bug-146178), noodles, bigjools, sinzui] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews
=== EdwinGrubbs is now known as Edwin-lunch
=== salgado is now known as salgado-afk
=== mwhudson_ is now known as mwhudson

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!