/srv/irclogs.ubuntu.com/2010/01/14/#launchpad-dev.txt

adiroibanHi! If a template is used in more views, should I define an interface for the template?01:58
jmladiroiban, an interface for the template?02:01
jmladiroiban, I'm not sure what that means.02:01
adiroibanall views using that template, to implement a common interface02:02
adiroibanright now we have potemplate-index.pt, potemplate-chart.pt, distroseries-langchart.pt and serieslanguage-index.pt02:03
adiroibanthey are for different object, but the statistic tables is similar02:04
adiroibanI was thinking those templates chould be changed to include a macro for that table02:05
jmlsorry, I zoned out...02:11
jmlumm...02:11
jmlthumper, ^^^02:11
thumperif the view is the same on all the pages, then yes, register against an interface02:12
thumperif it is similar but not the same, then use macros02:12
adiroibanthey are similar and I start using macros, but I was wandering if we should have an interface for that macro02:14
adiroibanyou can always read the macro code and see how it is accessing the view02:15
adiroibanbut I was just wondering if there is a convention for describing a macro.02:16
adiroibanthanks!02:16
thumperI have my own conventions for macros02:24
thumperbut I don't think there are documented and agreed upon conventions for LP devs02:24
jmlthumper, fight!02:31
bigjoolswgrant: Thou spleeny shard-borne bugbear!02:59
mwhudsonjml: http://uncyclopedia.wikia.com/wiki/Daily_Mail03:00
mwhudsonlanding branches throught pqm takes fricking ages03:13
magciusQuestion time: if I work on Code Hosting for Git branches over the next two weeks, will it ever get landed?03:15
magciusBecause I only want to try and attempt if it will be worthwhile.03:16
magciusSo basically: will Canonical consider other VCSs for Launchpad, because I have an amazing idea that I really want to see integrated, and I'd be glad to work on it, only if it would go somewhere.03:18
magciusBasically: tool-agnostic code hosting. Check out any branch in either Git or Bazaar and work on appropriately.03:19
thumpermagcius: what are you actually proposing?03:19
magciusthumper, what do you mean?03:20
thumpermagcius: you say you want to integrate git, but how are you proposing doing this?03:20
thumpermagcius: did you see jml's comment above?03:20
magciusthumper, which one?03:20
magcius<jml> magcius, we aren't opposed to having git support, but we'd want it to work in such a way so that you could use bzr or git for any project on Launchpad, regardless of what it's actually hosted in.03:21
magciusThat one?03:21
thumperyes03:21
magciusThat was basically my idea.03:21
magciusBasically, implement a git daemon that accepts pushes.03:21
thumperhow about log, diff, info et al03:22
magciusthumper, what do you mean?03:22
thumperI guess the real answer right now is no.03:22
thumpernot at this stage03:22
thumperI'm not saying we wouldn't consider it later03:22
thumperbut right now we have too much else on03:23
magciusthumper, okay.03:23
magciusthumper, the real work would be in converting the object database from bzr to git, right?03:24
thumperwhat object database?03:24
thumperwe use postgresql03:25
magciuserr03:31
magciusI'm thinking too much in terms of git.03:31
magciusI thought Keybuk told me that git and bzr were a lot closer in their storage model, but I guess that's not true.03:31
james_wsemantically close, but the details of what is stored differs, e.g. bzr doesn't store full file modes, and git stores less commit metadata03:33
james_wso a lossless mapping is difficult, if not impossible.03:33
=== stub1 is now known as stub
magciusjames_w, hm, okay.03:35
rockstarmagcius, out of curiosity, what is your favorite thing about Launchpad that you'd be so committed to getting Git support in.03:35
magciusrockstar, the fact that it could be a central place to do everything.03:36
magciusrockstar, as an advantage, Launchpad has the best bug reporting I've seen03:36
rockstarmagcius, yes, but there are many competing code hosting solutions out there that you could use.03:36
rockstarmagcius, awesome.  That's helpful to know.03:36
magciusrockstar, to be brutally honest, I'd be content with some way for the Branches thing to link to my GitHub repo.03:36
rockstarmagcius, you can say that code is not stored on Launchpad.03:37
magciusrockstar, my users were getting confused. Because I had forked a project already on Launchpad, they were like "huh? Where's the source?"03:38
magciusrockstar, I could provide an alternate bzr branch, but that's quite useless when it gets lost among the multitude of branches already there.03:38
rockstarmagcius, I think that's a realistic option in the short term.  Launchpad already has the concept of "remote branches."  Maybe that can be extended to point to git branches as well as bzr branches.03:39
rockstarmagcius, but you lose a lot of the niceties that Launchpad provides.03:39
magciusrockstar, I would also like it if I could fork a project so that I could fix bugs in my project's branch, and still get new bug reports from the old project.03:40
rockstarYou're using the term "fork" in the git sense of the word, correct?03:40
magciusrockstar, because one bug request of "fixed in this branch" that won't ever get integrated into the official one because it's a design decision is disappointing.03:40
magciusrockstar, basically, I forked notify-osd because I wanted to fix some things that were marked as wontfix.03:41
rockstarmagcius, well, to be truthful, as a user, if I wanted the same changes, I probably still wouldn't use your changes because I don't want to have to deal with two different VCSes.03:42
magciusrockstar, I initially added a branch and marked the bugs as "fixed in my branch", but Marco reverted that status to "wontfix"03:42
magciusrockstar, I initially hosted them with bzr, but when I started using git for my own projects I hosted it there too.03:43
thumpermagcius: why not have a vcs-import of your git branch?03:43
thumpermagcius: that'll stay in sync03:43
magciusthumper, because I'd like native support for Git. I have free time and I need a project to work on.03:44
thumpermagcius: native git support will not happen in the short term for launchpad03:44
thumperfull stop03:44
rockstarmagcius, I have some projects here you cane have.  :)03:44
magciusthumper, two questions: what time frame does "short term" mean, and why?03:45
rockstarmagcius, probably at least the next year == short term03:45
magciusAlright.03:45
rockstarmagcius, the why is because I could stop working on features now, and have at least a year's worth of work in bugs.  :)03:45
magciusWhen is the release date for "4.0"03:46
magciusrockstar, alright.03:46
magciusrockstar, I guess I should start smaller. I'd gladly fix the bugs.03:46
magciusjames_w, I was trying to find some good documentation on the formats and the object model, but I couldn't find any.03:46
thumpermagcius: awesome03:46
thumperbugs.launchpad.net/launchpad-project03:46
rockstarmagcius, I'm sure bzr-git could use your help.03:46
magciusrockstar, alright.03:46
rockstarmagcius, ignore thumper, he's a super crazy.03:46
magciusrockstar, are you a Launchpad dev or a bzr dev?03:47
* thumper sups his coffee03:47
rockstarmagcius, Launchpad dev, but I (as well as thumper) work on integrating bzr into Launchpad.03:47
magciuseek03:47
magciusrockstar, is Loggerhead a Canonical project?03:47
bigjoolsthumper the oompahloompah03:47
thumperoompahloompah?03:48
thumperI'm not that short03:48
jmlmagcius, kind of.03:48
thumperloggerhead is open source03:48
jmlmagcius, it wasn't started by Canonical03:48
rockstarmagcius, loggerhead didn't start at Canonical, but we loves it.03:48
jmlmagcius, and we use in for Launchpad, so we patch it a lot.03:48
magciusWell.. bzr didn't start at Canonical either :P03:49
thumperwell, yeah it did03:49
jmlmagcius, well, what do you mean by "Canonical project"?03:49
magciusjml, is it funded by Canonical? Worked on by Canonical employees?03:49
rockstarmagcius, I think you could provide some real help on bzr-git: https://bugs.edge.launchpad.net/bzr-git03:49
jmlmagcius, as I said, we patch it a lot.03:49
rockstarmagcius, no, but some of those who work on it are Canonical employees as well.03:50
magciusrockstar, alright.03:50
magciusrockstar, make Loggerhead easier to use. Every time I try to use it it's a cludge.03:50
rockstarmagcius, yes, we're working on it.03:50
magciusrockstar, make it have the Launchpad masthead with a branch breadcrumb03:50
magciusI guess I need a good overview on how bzr works internally, which I can't find online. All the documentation I can find is about bzrlib, which I don't care about.03:51
rockstarmagcius, yeah, like I said, year's worth of bugs if we stopped getting features. :)03:51
thumpermagcius: rockstar is somewhat busy on other tasks right now03:51
magciusSorry if that sounded like a demand.03:52
rockstarmagcius, bzrlib is how bzr works internally03:52
magciusrockstar, well, what's the object format?03:52
thumpermagcius: the best way to get bzr internals is to ask on #bzr03:52
rockstarmagcius, if appeal to thumper's emotions, maybe he'll make me work on Loggerhead.03:52
magciusthumper, okay. Is there a bzr-dev03:52
rockstarI doubt it though.  The Grinch's heart is bigger.03:52
magciuserr03:52
rockstar:)03:52
magcius#bzr-dev03:52
thumpermagcius: no, they are all on #bzr03:52
magciusokay03:53
=== thumper is now known as the-watcher
=== the-watcher is now known as thumper
jelmerthumper: I've updated the Branch destructor web API call branch again04:01
thumperjelmer: ok04:01
jelmerthumper: but something else seems to be wrong with the diff now04:03
jelmerI was just looking - there's a lot of other changes, but I've only merged lp:launchpad04:03
thumperhmm...04:30
stubjelmer: lp:launchpad is lp:~launchpad-pqm/launchpad/db-devel04:35
jelmerstub: Ah, that explains it - thanks04:38
jmlgah, my use-testtools branch _just_ missed the buildbot boat05:36
jmlnow I won't get it on stable for 8.5 hrs.05:36
lifelessjml: grats05:43
lifeless[on getting it in at all]05:43
jmllifeless, thanks, although I don't count a branch as landed until it hits stable.05:49
lifelessjml: tribunal needs an update for testtools - it doesn't upcall in setUp :P06:03
jmloops.06:04
* jml adds a todo06:05
stubjml: Perhaps we should have multiple builds running instead of just one.06:28
stubjml: We could have 4 slaves that kick off a build of trunk once per hour if there are revisions landed that are not already being tested by another buildbot.06:30
jmlstub, that's a good idea06:31
stubI think if you wanted to guarantee 15 mins max before a buildbot starts testing your patch and had a 4 hour test suite we would need 16 potential slaves configured06:35
* jml notes that PQM takes 15 mins to do its thing06:38
jmlback from nzpug?08:11
=== jml changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 1 of 10.01 | PQM is open | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in #launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | Channel logs: http://irclogs.ubuntu.com/
wgrantjml: We've been back for a while (it finished more than 1.5 hours ago)08:16
wgrantjml: But most were out for a while after that.08:16
mwhudsonevening08:17
jelmer'lo08:18
jmlwgrant, I see.08:19
bigjoolsha straight on the laptops I see08:20
james_wwhat else?08:22
jmlengaging conversation, a stroll on the waterfront, a round of whist by the fire...08:22
bigjoolsor something by the fire08:23
mwhudsonjml: i can tell you haven't been outside for a while08:23
mwhudsonit's raining now08:23
jmlmwhudson, heh heh08:23
jmlhttp://paste.ubuntu.com/356445/ -- what's going on here?08:24
mwhudsoni guess you have to edit the prepare script?08:24
mwhudsonjml: btw, if you want to look at https://code.edge.launchpad.net/~mwhudson/launchpad/recipe-model-code/+merge/16272 and suggest stuff about filenames and so on, feel free08:25
jmlmwhudson, edit it how?08:26
jmlmwhudson, I'll take a look08:26
mwhudsonta08:27
jmlmwhudson, I think that all of it could be just as easily in lp/code/ as lp/soyuz.08:29
mwhudsonyes i guess08:29
jmlmwhudson, I'm not sure which is better.08:29
wgrantAlso, there's that one thing which is in a file that is named strangely.08:29
wgrantrecipebuilder.py08:29
mwhudsonjml: it's a fascist conspiracy that file paths are linear08:29
jmlmwhudson, indeed.08:29
mwhudsonwgrant: that's not in my branch, at least :-)08:29
wgrantmwhudson: Ah, OK.08:30
jmlleonardr, http://paste.ubuntu.com/356445/ is what happens when I try to run 'prepare' in lazr.yourpkg08:35
james_wjml: do you have any lazr packages elsewhere in sys.path?08:37
jmljames_w, quite probably.08:37
jmljamesh, sorry.08:38
jmlgror08:38
* jml confused08:38
jameshhmm?08:38
leonardrjml: i think you're supposed to branch it as 'lazr.yourpkg' and then run 'prepare' to change it everywhere08:39
jmljamesh, sorry, I got your nick confused w/ james_w as sort of a double-misunderstanding08:39
jmlleonardr, that also doesn't work08:40
jmlleonardr, http://paste.ubuntu.com/356450/08:40
leonardrjml: ok, i'll try it08:40
jmlleonardr, thanks.08:40
leonardrjml: ok, you don't have to branch it as lazr.yourpkg08:42
leonardrbut you do have to run bootstrap.py and then bin/buildout08:42
leonardrthen you can run bin/py prepare.py lazr.importguardian08:43
jmlleonardr, thanks.08:54
jmlleonardr, were there docs in the package that I was missing?08:54
leonardrjml: not really, you're just supposed to know how to run lazr.* scripts from https://dev.launchpad.net/HackingLazrLibraries08:55
jmlleonardr, ahh ok.08:55
leonardrjml: but yourpkg is kind of a special case, it's not obvious that you have to run buildout on a package you're not going to use08:56
leonardrif you wanted to add some more specific docs to README i'd r= that08:56
jmlleonardr, thanks.08:57
mrevellMorning09:10
jmlmrevell, hi09:12
mrevellhey there jml09:12
jmlleonardr, even after bootstrap & ./bin/buildout, I still get the same error09:12
leonardrjml: just to check, are you using bin/py?09:13
jmlleonardr, yes.09:14
leonardrhmm09:14
leonardri got it to work09:14
jmlleonardr, this is in the lazr.importguardian branch. trying in lazr.yourpkg now.09:14
jmlI wonder if there's a correlation between liking buildout and being close to the US east coast (where the internet lives)09:15
jmlnope. :(09:16
leonardrok, let's try it again09:16
leonardrjml: yes, i'm getting an error now in buildout. you too?09:22
leonardri think there is a catch-22 here09:22
wgrantjames_w/abentley: Hm, I see that the slave returns a file named 'changes'.09:23
wgrantwhy isn't it the full name?09:23
jmlleonardr, http://paste.ubuntu.com/356466/09:23
jmlleonardr, that's what the first buildout run gives me09:23
wgrantI could hack around that, but it would be preferable to have the full name returned.09:23
leonardrjml: do you have a global cache? you might like buildout better if you set one up09:24
leonardrthat's also in HackingLazrLibraries09:24
james_wwgrant: oh, I noticed that, but I thought it was intended09:25
james_wwe can probably fix it09:25
wgrantjames_w: I can't see where it is actually done on the slave side.09:25
abentleywgrant: That's what's done for a binary build.09:26
jmlleonardr, ahh, that's a useful tip. thanks.09:26
leonardrjml: and are you sure those errors are fatal? is there a bin/py in your directory?09:27
jmlleonardr, they aren't fatal. there is a bin/py09:27
wgrantabentley: Oh, yuck. I guess I'll check how process-upload handles that, then. Best not to change the slave.09:27
jmlrunning again...09:27
james_wpsycopg2.OperationalError: FATAL:  Ident authentication failed for user "launchpad" <- any clues?09:29
wgrantjames_w: You've run launchpad-database-setup?09:29
james_wthis is the first time I'm running LP outside a chroot09:29
james_wyep09:29
jmlleonardr, http://pastebin.ubuntu.com/356471/ is what I end up with in ./bin/py09:30
jmlleonardr, I also have lazr packages installed on my system from Ubuntu.09:30
jmlleonardr, should I be using virtualenv?09:31
leonardrjml: i'm not sure. i eventually had to compile a whole separate version of python. if you run into problems or are already good at virtualenv, i'd recommend it09:33
wgrantabentley: That's only done for a binary build as of your r10153.09:33
wgrantabentley: Previously it used the real name.09:33
wgrantNow I suspect it is broken.09:33
abentleywgrant: Okay, I'll have a look at it tomorrow.09:34
wgrantabentley: Thanks.09:35
jmlleonardr, I'm not :(09:35
jmlleonardr, the HackingLazrLibraries page says there are issues that are being looked into -- is there a relevant bug number?09:35
james_w275+        self._slave.waitingfiles['changes'] should be self._slave.waitingfiles['09:36
jml"At this time, a non-system Python, or a virtualenv executable that does not include site-packages, is highly recommended, and may be required. We are working to remove this limitation."09:36
james_wself._slave.waitingfiles[os.path.basename(path)] I mean09:36
leonardrjml: i don't know about this part, sorry09:36
jmlleonardr, ok, thanks.09:36
wgrantjames_w: Yeah, that's what I've just hacked in.09:37
wgrantjames_w: If one of you can commit that to your branch, 'twould be good.09:37
leonardrjml: you might ask gary or (for some reason i associate him with this) bac09:37
jmlleonardr, is there a lazr mailing list?09:38
leonardrjml: yeeees?09:39
leonardrhttps://edge.launchpad.net/~lazr-users09:39
jmlleonardr, thanks.09:39
* jml is off09:45
jmlg'night all.09:45
wgrantOooh.09:51
wgrantCollection works.09:51
james_wyay09:54
wgrantI have a lot of local changes to bash SourcePackageRecipeBuild into its interface, but it works.09:55
mwhudsonwgrant: woo10:48
wgrantmwhudson: Took a bit of work, but I successfully (and automatically) went from the SourcePackageRecipeBuild that you told me how to create to a binary package, with not too many gross hacks.10:50
* wgrant decides to sleep.10:50
mwhudsonwgrant: very cool10:51
* mwhudson should sleep too10:51
mwhudson(and mthaddon has built buildbot two of the ec2 images already, hurrah!)10:51
wgrantSo we can land stuff tomorrow?10:51
deryckMorning, all.10:58
=== matsubara-afk is now known as matsubara
=== mrevell is now known as mrevell-lunch
kfogeladeuring: what does the "structure" keyword in, e.g., <a tal:replace="structure patch/bug/fmt:link" /> mean?  It solved a huge problem for me (my output links are correct now, instead of looking like quoted HTML), but I'm not sure what magic did this...13:19
adeuringkfogel: it prevents escaping of '<' and '>'13:20
adeuring...in the text that is inserted13:20
kfogeladeuring: aaaah, "structure" as in "interpret this as the 'structure' of an HTML page instead of as content" ?13:20
adeuringkfogel: yes13:20
kfogeladeuring: interesting.  Very useful; not the keyword I would have chosen, but then I guess it's hard to think of words that will be intuitive under all circumstances.  Anyway, thank you.  Big light bulb for me :-).13:21
adeuringkfogel: welcome :) The general idea is: Use "structure" only for trusted data; any user provided text must not be rendered this way.13:21
kfogeladeuring: *nod*  thanks.  In this case, it's expanding our own bug link, so it's safe (though users can set the titles of bugs... I presume we already filter that in the bug tracker itself?)13:22
adeuringkfogel: well, the title is considered as text, so it should not be rendered with the "strurcture" keyword. That's even necessary, otherwise writing a title like "1<2 returns False" would be a bit cumbersome13:24
kfogeladeuring: one sec, let me show you diff and screenshot13:25
adeuringkfogel: and sure, using "structure" to render a link is absolutely fine -- that's the indended usage13:26
adeuringkfogel: escaping the title is in this case the job of the fmt.link renderer13:26
adeurings/fmt.link/fmt:link/13:27
kfogeladeuring: http://paste.ubuntu.com/356555/13:28
kfogeladeuring: so that's a correct use of structure?13:28
adeuringkfogel: yes13:28
kfogeladeuring: thanks13:28
adeuringkfogel: welcome :)13:28
=== mrevell-lunch is now known as mrevell
marsmorning flacoste!14:37
flacostemorning mars!14:38
=== matsubara is now known as matsubara-lunch
=== salgado is now known as salgado-lunch
kfogeladeuring, deryck, intellectronica: what free software tools do you like for making mockups?  I can just use inkscape or gimp or whatever; I'm just doing a rough sketch for Bryce right now, and wondered if you had any tools you love.15:22
deryckkfogel, with image-based mockups I use gimp.  I use balsamiq (not free) for free-from, sketch-type mockups.15:23
adeuringkfogel: pencil, paper, scanner and Sane?15:23
kfogeladeuring: :-) could work15:23
adeuringkfogel: problem is though that only Sane is free in this setup...15:24
intellectronicakfogel: inkscape is quite good for sketching. if you're willing to compromise on freedom balsamiq is very good and we have a group license for it (given by balsamiq gratis to free software projects)15:25
kfogeladeuring: they're all libre.  in fact, I think I libre-ated some of my pencils from the Software Freedom Law Center a few weeks ago.  Don't tell.15:25
kfogelintellectronica: I don't have a moral opposition to using non-free software, I just hate spending time acquiring skills in something that can be taken away at any time, if you know what I mean -- seems like a poor investment usually.15:26
adeuringkfogel: ;) And you can take paper from your recycling bin. Would also be free in some sense...15:26
intellectronicakfogel: yes, it's a fine balance. and even balsamiq is not free of problems. it's built on adobe air, which doesn't work very well on linux (shortcut keys don't function) and there's absolutely nothing you can do about it15:28
marsintellectronica, "shortcut keys don't function" - really?  All of them?  They've all worked for me so far.15:38
intellectronicamars: oh really? maybe something is borked about my installations (more than one, both in jaunty and karmic) but ctrl-c, ctrl-v, ctrl-x don't work, which is a pain in the ----15:40
marskfogel, you can export balsamiq mockups as .png for sharing, and their interface makes it very easy and streamlined to do so (easy to use UI from a UI design product company - go figure)15:41
marsintellectronica, those keys work fine for me here - GNOME on Karmic15:42
intellectronicahmmmm ... not that i have the time or patience to try fixing that now, but at least there's hope! :)15:43
marskfogel, if you go for Balsamiq and are running a 64bit OS, I have some simple instructions for getting AIR running.15:45
kfogelmars: thanks15:45
=== matsubara-lunch is now known as matsubara
matsubaraChex, gary_poster, rockstar, daniloff, sinzui, allenap: meeting in 1 min @ #launchpad-meeting15:59
matsubarahenninge, can you cover for danilo?15:59
=== beuno is now known as beuno-lunch
=== salgado-lunch is now known as salgado
=== henninge_ is now known as henninge
EdwinGrubbssinzui: what does the SourcePackageRelease.component column mean?16:55
sinzuiEdwinGrubbs: main, universe, multiverse16:57
sinzuiEdwinGrubbs: Main is of course the packages most often used by Ubuntu users16:57
=== beuno-lunch is now known as beuno
EdwinGrubbssinzui: shouldn't you limit which bugs you count by the date they were created? If a package is obsolete it should be significantly less important.17:10
sinzuiEdwinGrubbs: are any obsolete packages in the listing? I was assuming that they way they are joined they are current (latest release)17:12
EdwinGrubbssinzui: I don't know enough about those tables to say. Even if they are not to-be-removed from Ubuntu, they may be less important because a new piece of software is much better.17:14
sinzuiEdwinGrubbs: I suppose `spph.supersededby is not NULL` would ensure the that the package is current17:15
sinzuiEdwinGrubbs: when a package is removed from a distroseries, the spr is removed, which is what causes the message in the sp +index that the package is not in the series17:16
sinzuiEdwinGrubbs: I think the status of published means it is current so a constraint like `spph.status = 2` will work17:20
EdwinGrubbssinzui: what is the most important thing that will happen after these items are linked? Build a new binary package in Ubuntu or in a PPA?17:25
sinzuiEdwinGrubbs: ppa are not used in buildin ubuntu.17:28
sinzuiEdwinGrubbs: The most important thing is to then ensure the branch, bugtracker, and translations are setup so that bugs are forwarded, translations synced, and tip is included in the daily builds17:29
EdwinGrubbssinzui: it just seems odd that nagios-plugins would be ranked higher than sun-java6 and postfix.17:35
sinzuisun-java is not in main.17:36
* sinzui looks for popcon data17:37
EdwinGrubbssinzui: it seems that buggier apps will get prioritized over those with a lot of bug heat for a few bugs.17:38
sinzuiEdwinGrubbs: yes. I do not want to write queries that will kill the page (just like i did in November)17:39
sinzuiEdwinGrubbs: nagios-plugins are pretty popular17:40
sinzuiEdwinGrubbs: I am not sure heat in needed to this prioritisation to be successful.17:41
EdwinGrubbssinzui: the heat was just an idea, but I don't think it's that important. I do think that a component package with zero bugs and zero translations shouldn't win out over a non-component package with bugs or translations.17:43
sinzuiEdwinGrubbs: yes, I saw that. the main rule was that last thing I added.17:44
EdwinGrubbssinzui: otherwise, I think the heuristics are good. It would probably make it easier to update the query if you put it in a subquery so that you could use "ORDER BY score DESC"17:45
sinzuiEdwinGrubbs: instead of counting the bugs, I could sum the bug.hotness, but the default is 0 so I need to verify 1 is the effective value since there is always on subscriber17:45
sinzuiEdwinGrubbs: great idea, I'll try that17:46
sinzuiEdwinGrubbs: r=me for the API branch. I have one suggestion in my review.17:50
EdwinGrubbsthanks17:50
sinzuiEdwinGrubbs:  the two packaging queries use spph.status IN (1, 2) which means the package is pending or published. There are no deleted or obsolete packages in the listing17:52
sinzuiEdwinGrubbs: Switching to sum(hotness) instead count(bugtask.id) creates an interesting change in the top 15 (I used LIMIT): http://pastebin.ubuntu.com/356690/18:13
mrevellnight18:14
=== deryck is now known as deryck[lunch]
mwhudsonmorning19:10
wgrantMorning.19:11
=== deryck[lunch] is now known as deryck
EdwinGrubbssinzui: that is interesting. maybe, it should be (hotness/10)+count so that desktop apps don't have such an advantage over libraries.19:21
sinzuiEdwinGrubbs: I'll try that. I have already changed main component to 1000 and increased the po_message core by a multiple of 519:22
beunomwhudson, hey. When's the LH patch going to be tested?   I'm anxious to see if a 20 line patch fixes everything  :)19:35
mwhudsonbeuno: it should automagically hit production "at some point"19:35
beunowe should really stop using voodoo to deploy launchpad19:36
mwhudsonlooks like it's rolled out at 22:30 utc19:37
mwhudsonso ~3 hrs19:37
* beuno crosses fingers19:38
beunokfogel, another alternative to show the patch-er19:57
beunois to show the one with most karma19:57
beuno(most active in LP)19:58
kfogelbeuno: interesting idea!19:59
kfogelbeuno: I'm a little worried about karma becoming a positive feedback loop; but, we could at least show karma when we show a user at all.19:59
thumpermorning20:01
=== matsubara is now known as matsubara-afk
=== salgado is now known as salgado-afk
kfogelintellectronica: ayt?20:12
al-maisanwgrant: Could you please take a look at comment #2 in https://bugs.edge.launchpad.net/soyuz/+bug/50732320:13
mupBug #507323: Candidate job selection logic needs to observe the 'virtualized' setting <buildfarm> <Soyuz:In Progress by al-maisan> <https://launchpad.net/bugs/507323>20:13
kfogelderyck: got time for a quick call about code design?20:17
kfogel(not vital, just would help; I can stumble through the underbrush on my own too)20:17
kfogelbeuno: have you seen the "mockups" in bug #506018?20:20
mupBug #506018: Need a "+patches" view: report lists patches attached to bugs. <story-patch-report> <Launchpad Bugs:Triaged by kfogel> <https://launchpad.net/bugs/506018>20:20
beunokfogel, I have. I've been spying on the thread20:21
beunothe "..." may not be the most obvious and clickable thing  :)20:22
kfogelbeuno: yeah.  Maybe the words "(most recent)" next to the patch age?  (and the words would not be a link)20:22
kfogelsometimes one can try to do too much with icons!20:23
beunokfogel, I think I'd just not link it20:23
beunoand20:23
beunoI've also been thinking about this number20:23
kfogelbeuno: the patch age?20:23
beunoinstead of days/weeks/months20:23
beunoyes20:23
kfogelwe need to indicate units20:23
kfogelright now I'm assuming it's days20:23
kfogeluser may not know that though20:23
beunoright, I'd add that explicetely20:23
kfogelagreed20:24
beunomaybe for multiple patches20:24
beunoyou could do:  61 days (2)20:24
kfogelshould the patch age itself be a link to anything?  I'm beginning to think not.  It should just provide the hover-over popup.20:24
kfogelbeuno: that'd be confusing, though -- is it saying there are 2 patches that have been there for 61 days?20:24
beunoI think that unless you take them directly to the patch or something, no link20:24
kfogelthat's not the case20:24
beunoit would sort of imply that20:25
kfogelwell, going directly to the patch, hmmmm... that's an idea.  We provide a snippet of it in the popup, then they can click directly to it to investigate more.20:25
beunonot sure it's terrible20:25
kfogelhow about "61 days (youngest)" ?20:25
beunowell, how important is it to know that the other ones are older?20:26
kfogelIt's somewhat useful to know there are other patches there, but it may not be worth the UI clutter.20:26
kfogelBryce seems to think not hugely important.20:26
kfogelWe can just leave it out for now.20:26
kfogelWe're going to talk to an upstream dev too (jorge is finding) so we can ask him.20:26
kfogelBut for now I'll code assuming we just show the youngest and that's that.20:26
beunoI think it's good to know there are more patches, but not super necessary to know that the "other ones" are older20:26
beunoadding the number between brackets is not super correct, but helpful enough  :)20:27
kfogelbeuno: true, though that will always be the case20:27
beunoand don't show it if there's just one  :)20:27
kfogelumrmph20:27
kfogelI am afraid that number is going to be confusing.  let's see what bryce & upstream dev say20:27
beunosure20:27
beunokfogel, I'd also link the packages to their bug listings20:28
beunoand, of course, importance/status with the colors20:28
kfogelbeuno: IOW, to the same place the bug number & summary link to?20:28
beunokfogel, no, the bug will take you to that bug, this will take you to the list of open bugs for that package, or hot bugs, your call20:29
kfogelwhat do you mean by "important/status with the colors"20:29
beunoas a way to go to that package20:29
beunokfogel, colors, https://bugs.edge.launchpad.net/bzr20:29
beunoconfirmed red, fixed greed, etc20:29
kfogelbeuno: oh, of course.  Bryce's page isn't already doing that with the colors?20:30
* kfogel looks20:30
kfogelheh, it's not20:30
kfogelBut I assumed it was, no worries.20:30
beunocool20:31
beunoI'll continue to spy, and try and raise things before it hits UI review20:31
beunokfogel, it rocks that you're doing this20:32
kfogelbeuno: I'm loving it, but WHEW it is really drinking from the firehose.  This isn't just a minor bugfix :-).  I'm learning a lot about launchpad plumbing.20:33
jml:D20:34
jmllaunchpad needs more plumbers20:34
beunokfogel, you should try something with blueprints  :)20:34
kfogelbeuno: or maybe I should just slam this steel door on my toes?20:34
beunokfogel, good, you got the picture20:35
kfogeljml: want to discuss a code-design question with me?  Nothing better to do?20:41
jmlkfogel, "better" is an open question. I do have stuff that's more urgent, sadly.20:42
kfogeljml: thought that might be the case, no worries.20:42
beunosinzui, is karma updated only once a day?20:53
* kfogel discovers http://www.owlfish.com/software/simpleTAL/tal-guide.html, weeps with joy21:01
thumperbeuno: if that21:02
beunook, once-a-day graph should do then21:03
sinzuibeuno: karma is once a day, and revision karma only when the script completes21:08
beunosinzui, thanks21:09
sinzuikfogel: yes, that is the same page I bookmarked when I started hacking on launchpad.21:18
jtval-maisan: https://bugs.edge.launchpad.net/rosetta/+bug/50767821:55
mupBug #507678: Retrieve TranslationTemplatesBuildJob results <Launchpad Translations:New> <https://launchpad.net/bugs/507678>21:55
al-maisanjtv: thanking you :)21:55
jtval-maisan: no, you!21:56
kfogelsinzui: in one of our .pt files, I'm still not clear on where the "context" comes from in something like this: <tr tal:repeat="patch context/patches">22:14
kfogelsinzui: I mean, it's working, I just don't understand all the magic.  Can you summarize what the "context" is?22:14
sinzuicontext is the context of the view, and if the template is used by many views, the context could be several objects that implement a common intergface22:15
sinzuikfogel the View that does the work for the template is __init__ ed with the context object and the request. The context was passed to the view by the published (in th case of the main view seen in the url), or by another template using object/@@/a-view22:16
sinzuikfogel: I think the context in your specific case is BugMessage because the model object has a patches attribute22:19
kfogelsinzui: HasBugsBase (the patches attr is new in my branch), but thank you -- I think I get the idea.  I was unclear on how context was magically what I needed it to be :-).22:27
sinzuikfogel: context and view are the common objects we work with in templates. What is very confusing is there is a CONTEXTS that is something very different. there are a few templates that use it. Try to ignore it if you can22:29
kfogelsinzui: school of hard knocks: I just learned you can't use a "-" in the NAME portion of a tal:define="NAME EXPRESSION".  Now *that* was unexpected.23:31
sinzuiI think we have all done that. You have graduated23:31
kfogelsinzui: stumbled across it early in a lucky guess; I wouldn't call myself bitter.  Yet.23:33
sinzuikfogel: When you work with CSS3 or YUI, you learn you cannot use "_" or "." in a css class name, but many of our templates are doing that, so you need to fix the class to use it in JS.23:33
kfogelwhoa23:33
kfogelsinzui: thanks for the heads-up!23:33
BjornTsinzui, kfogel: you can select by class containing dots in yui. you just have to use attribute selectors (like '[class="some.name"]'23:44
sinzuiBjornT: right, and that is something you learn the hardway23:45
kfogelBjornT: *nod*  I'll remember at least to ask you when I encounter that :-).23:45
=== jamalta_ is now known as jamalta

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