/srv/irclogs.ubuntu.com/2012/12/13/#ubuntu-unity.txt

smspillazduflu: hollyyyy ... did you see the new nvidia driver release ?02:17
smspillazImproved performance of OpenGL framebuffer object binds with Xinerama enabled by 2000-3000% when the application's windows do not span screen boundaries.02:18
smspillazlol02:18
dufluHah02:19
dufluNice02:19
smspillazduflu: buffer age support too!02:19
* smspillaz cracks open his branch to try it out to see how much faster it is02:19
smspillazduflu: buffer_age means that we can finally use glXSwapBuffers without redrawing the whole backbuffer on every frame02:22
dufluThat's handy, but not if only 1 or 2 drivers do it02:22
duflusmspillaz^02:22
smspillazstill great if at least one does!02:23
smspillazthe free drivers will support it in the next kernel release from what I've heard02:23
duflusmspillaz: Mesa claimed to implement GLX_OML_swap_method but it never really worked. So I'm not holding my breath02:24
smspillazduflu: buffer age is different02:24
smspillazits really easy to implement driver side02:24
dufluMy point still holds. I'll believe it when it's released and working02:24
smspillaz:)02:24
* smspillaz adds xorg-edges02:25
smspillaz*xorg-edgers02:25
smspillazduflu: seems to work OK, though I suspect their implementation is a bit slow atm03:42
smspillazmaybe they do the buffer age tracking server side03:42
smspillazduflu: nope, scratch that, forgot to enable lazy positioning :)03:55
smspillazduflu: so awesome, it works great. I'll have something up for you /others to test tomorrowish03:55
duflusmspillaz: Tomorrow is my last day before vacation. No promise of any more large reviews this year :/03:58
smspillazduflu: no problem, it needs lots of testing anyways, it can wait04:00
smspillazduflu: anyways, we no longer need to worry about the performance impact of doing that fbo bind on every frame, at least on nvidia \o/04:01
smspillazI think nvidia is going to go from our worst performing driver to our best04:01
duflusmspillaz: Cool. I thought I modified the FBO class to be smart enough to never rebind until absolutely required though04:01
duflu(when it needs a different FBO)04:02
smspillazduflu: ah, when was that ?04:02
duflusmspillaz: Before GLES landed04:02
smspillazright04:02
smspillazduflu: althoguh you still have to switch between fbo and backbuffer to draw a frame04:02
duflusmspillaz: But that is only optimal for that class. If you can make the calling code (opengl plugin) rebind to different FBOs less often then that could still help04:03
smspillazindeed, although for most frames we only do it once04:03
smspillazerm, twice04:03
smspillazonce to fbo, once to back04:03
smspillazglxswapbuffers04:03
smspillazwith buffer_age we don't even have to do that04:03
smspillazI wrote some code a few days ago to track damage regions across old backbuffers, so gl tells you how old (in frames) your backbuffer is and then you just repair the relevant regions04:04
duflusmspillaz: Also worth checking Nux to see if and where it does its own rebinds. I suspect it does too much but don't really know04:04
smspillazduflu: it does do way too much :(04:04
duflusmspillaz: Which history says is a significant bottleneck :(04:05
duflusmspillaz: I am curious because Nux+Unity works with all Compiz rendering modes now. If you're already using FBO then Nux should rebind less04:05
smspillazduflu: the best thing we can do04:06
smspillazduflu: is to determine when you're using a framebuffer object of the same size04:06
smspillazand then switch between the color attachments04:06
smspillazthats basically free04:06
smspillazit'll require a bit more infrastructure between nux and compiz04:07
duflusmspillaz: Nux seems flexible enough that it *should* be able to work without FBOs. I don't understand why it needs them at all. Just give it a backbuffer...04:07
smspillazduflu: you need fbos to do gaussian blurs04:07
duflusmspillaz: Oh, I forgot04:07
smspillazwell, you can do them without fbos but its much slower :)04:07
duflusmspillaz: I think it should be much faster. The need for FBOs in blurring is to only ensure there is no "drift" like you see with an in-situ blur04:09
dufluBut regardless, if Unity turns off active blur, that should in the least prevent Nux using FBOs. People should have the option of making things faster even if it's not the default04:10
smspillazduflu: also nux uses fbos becauses its convenient really04:10
smspillazeven though it shouldn't04:10
smspillazduflu: anyways, the thing with gaussian blurs is that you need to do the 2nd pass using the results of the first04:12
smspillazthere might be ways to have pseudo-gaussian blurs which don't use 2 passes, but at the moment we use 2 passes04:12
smspillazmultiple color buffers are the best way to do that - reading off the backbuffer is expensive04:12
duflusmspillaz: Yes true for Gaussian. However there other countless other algorithms that don't require multiple orthogonal passes.04:12
smspillazindeed04:12
smspillazgaussian looks the best though04:13
smspillazanways gottan run04:13
dufluYeah Gaussian looks best but in a real-time UI, people will never tell the difference04:13
dufluSorry, I don't mean to be one who abuses the term "real-time". I know Linux is not a real-time OS04:14
didrockssil2100: hey! when you get some time, do you mind updating me on the failing tests?08:37
sil2100didrocks: sure thing! I already have a branch prepared, when I'm done with this one more fix I will give it out for review - we this should fix 2 failing tests at least08:55
didrockssil2100: sweet! do you think we'll be able to get down to 0 from this list failing?08:56
didrockssil2100: as it's only a subpart of tests for non regressing other stacks, I would love setting the value to 008:56
sil2100I would hope to get it down to 0, but some test failures are hard to track down, especially when they're hard to reproduce or some singular cases - I tried one of them yesterday and didn't yet find the root cause for the 'accidental' failure ;/08:59
didrockssil2100: maybe fghinter can give you access to the machine? as we are not using them, we can keep them in a fail case maybe so that it's easier for you to run them by hand?08:59
sil2100didrocks: hm, I guess it wouldn't mind to try, I'll ping Francis when he's online09:05
nikttohi all! I have one quick question about AppIndicator3 in python. I'm using it as an alternative to GtkStatusIcon and I'm connecting same GtkMenu to it but it seems that I can't catch 'button-press-event' on GtkMenuItem10:05
nikttois there any road around this without using 'activate' signal?10:06
smspillazduflu: hey, see my comment on the MRQ - I think the checks for nBounding < 1 and nInput < 1 are wrong and the source of the problem11:04
duflusmspillaz: Ah yes, good point. But we still want to support !XShape11:05
smspillazduflu: indeed, see my comment for an idea on how to do that11:05
dufluSo feel free to propose something that solves it11:05
dufluNight11:05
smspillazduflu: are you working tomorrow ?11:06
dufluYep11:06
smspillazduflu: ok, well, I guess you can probably handle it11:06
=== _salem is now known as salem_
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== dandrader is now known as dandrader|lunch
didrocksmterry: hey, thanks on the testapp MP! Feel free to get it merged, but I think we shouldn't automate it for daily process until we settle down on the name :)14:36
mterrydidrocks, yeah14:36
didrocksmterry: something else, I think you saw that libunity-misc is not merging :/14:36
* didrocks keeps seeing autolanding branch being stalled14:36
mterrydidrocks, oh?  wait I thought it did14:36
didrockshttps://code.launchpad.net/~mterry/libunity-misc/modernize-build/+merge/13949914:36
didrocksstill approved14:37
didrocksfginther: ^14:37
didrocksfginther: do you have anything to spot those branches? I keep having 3/4 a days in that stalled situation14:37
mterrydidrocks, ah right. But not rejected due to jenkins.  Just no jenkins around14:37
didrocksnot really efficient to have to ping you or mmrazik every time14:37
didrocksmterry: ah, something else (and sorry to add more to your plate), but there will maybe be some SRU on webapps for youtube which changed its layout, so the script broke14:38
didrocksmterry: robru will do the packaging, but he will need a sponsor next week I guess14:39
mterrydidrocks, OK14:39
didrocksso if you can have a look at it when it will be time :14:39
fgintherdidrocks, yes, we have job that checks for these. It's probably emailing marting, but not me...14:39
fginthers/marting/maring/14:39
fginthers/maring/martin/14:39
didrocksthanks mterry ;) FYI, I don't really know how the source packages are created (there is one source package for each script, but the real source is webapps-applications)14:39
mterrydidrocks, yeah, they generate them somehow.  also not sure how14:40
didrocksmterry: so for this security fix, mdeslaur and I just apt-get source and distro-patch until ken is back and we know how it works with their pkgme14:40
didrocks(the one we uploaded less than an hour ago)14:40
didrocksI guess you will have to do the same for the youtube one14:40
didrocksfginther: it needs patching then! :)14:40
fgintherdidrocks, mterry. sorry about that. It should be building shortly14:43
didrocksfginther: no worry, thanks! :)14:43
didrocksand one more on the autolanding list (even if the unity stack is not scheduled)14:43
mterrydidrocks, what's the blocker for the unity stack?  autopilot still?14:44
mterry(in terms of turning on autopublishing)14:44
didrocksmterry: yeah, we just got the first reliable results yesterday14:44
didrocksmterry: so, trying that + adding the dependencies between stacks (more on that in a publishing doc) once sil2100 will have few tests ironed out14:45
didrockswe'll probably do "one blank shot" tomorrow (without really releasing) with jibel14:45
didrocksbut I don't feel confortable enabling that just before going on vacations :)14:45
didrocksmterry: I think I'll disable all the jobs btw. I don't think that in what is bootstrapped and running already, we'll have emergency fix14:46
sil2100didrocks: yes, and I also probably found a bug in appmenu in the meantime as well14:46
didrockssil2100: \o/14:46
didrockslarsu: ^14:46
sil2100Trevinho is helping me out here as well14:46
didrocksoh sweet!14:46
fginthermhr3, can you review https://code.launchpad.net/~fginther/dee/add-code-coverage/+merge/139586 or suggest someone?14:47
larsudidrocks, what do you mean? The appmenu bug?14:47
didrocksmterry: not sure you were able to talk to cyphermox yesterday, I didn't get any feedback, did you have a chance to work with him on the failing tests in a chroot?14:47
didrockslarsu: not sure it's the same than Trevinho and sil2100 are looking at14:48
cyphermoxdidrocks: I was out cold all day yesterday14:48
didrocksso maybe better that the 3 of you sync!14:48
cyphermoxI poked mterry this morning14:48
mterrydidrocks, huh.  I can't mark the testapp branch approved.  Can you do that?14:48
didrockscyphermox: yeah, hello btw! do you feel better?14:48
cyphermoxdidrocks: a little, yeah14:48
didrocksmterry: do you want that power?14:48
didrockscyphermox: thé + miel? :)14:48
mterrydidrocks, sure (muhahaha!   more power!)14:49
cyphermoxat least now I can actually stick to one physical location at home... ykwim14:49
didrocksykwim?14:49
didrocksyou know what I mean14:49
didrocksI guess :)14:49
cyphermox;)14:49
didrockscyphermox: take it easy! :)14:49
cyphermoxmeh, things still need to get done, and it's been taking way too long already14:50
didrocksmterry: waow! you are an autopilot hacker now, impressive! :p14:50
didrocksfginther: mterry: libunity-misc merged now and adding to the bootstrapped list! awesome :)14:51
mhr3fginther, configure.ac:146: cannot open `build/autotools/gcov.m4': No such file or directory14:55
mhr3fginther, looks like it doesn't support non-srcdir build14:55
=== james is now known as Guest16278
fginthermhr3, thanks I'll fixt that14:56
=== dandrader|lunch is now known as dandrader
fginthermhr3, how are you doing the non-srcdir build?15:21
mhr3fginther, mkdir builddir; cd builddir; ../autogen.sh; make15:24
fginthermhr3, thanks15:24
didrockslarsu: sil2100: sweet a fix and perf improvments!16:00
larsudidrocks, the performance increase was rather tongue-in-cheek ... it's only 4 empty indicators not being transferred over the bus16:01
sil2100:)16:01
larsubut otherwise: yeah! I love how easy this was to fix16:01
didrockslarsu: ah ok, I thought it was for all exported menus :)16:01
sil2100I'm testing larsu's solution now16:01
larsudidrocks, yes, all exported menus (that are GMenuModels)16:01
larsudidrocks, 4 is the number for charmap, it's really the amount of toplevel menu items16:01
larsubut still, since the indicators are empty, there's not much data ...16:02
didrocksah ok, only toplevel menu indicators ;)16:02
didrocksyep16:02
larsuyeah but it's more Right now :)16:03
sil2100larsu: tested, works! Can I approve globally, or do you want someone else to also review it before that?16:14
larsusil2100, I want tedg to have a look as well, it's his code16:15
larsusil2100, thanks for testing!16:15
sil2100Would be awesome to have it merged in, I won't have to push the autopilot test-hack otherwise ;)16:16
Zhenech?window 3616:43
tedglarsu, Any clue on why GtkMenu inserts separators?16:56
tedglarsu, Seems kinda ridiculous.16:57
larsutedg, that's the only way to do separators with GMenuModel (have them between all toplevel sections/items)16:58
tedglarsu, Ha, okay.  Feeling sorry for who ever has to write appmenu-gtk for GMenuModel.  :-)16:59
larsuotherwise you'd never have any separators16:59
larsutedg, attente? :)16:59
larsuaww, he's not in here16:59
tedgAh, didn't realize someone was on it.16:59
tedgIt's going to be super painful as GMenuModel is so inflexible.17:00
larsutedg, I hear he's making good progress, though :) Let's see how it turns out17:00
larsutedg, thanks for approving17:01
larsusil2100, should land soonish17:01
=== salem_ is now known as _salem
tedglarsu, No problem, finding it is harder than approving it :-)17:01
larsutedg, right...17:02
* larsu was searching in unity-panel-service at first17:02
tedglarsu, I'm sure he will, just inferring what the app was trying to do is going to be tricky.  A lot of fakery.  Plus you have to deal with lots of things changing that GMenuModel doesn't allow.17:02
larsutedg, ya, when I first heard about it I thought it was an impossible job :)17:05
=== _salem is now known as salem_
sil2100\o/17:09
sil2100Thanks :)17:09
* larsu <-- dinner17:10
* tedg is a little concerned that larsu just got turned into dinner, but is hoping for the best.17:11
didrockstedg: you know, on those countries…17:12
* tedg is a little concerned that didrocks is going to be turned into wine, but is hoping it is a nice wine ;-)17:15
didrockstedg: who will be the cheese then? :)17:15
tedgdidrocks, seb128!17:15
seb128tedg, heh, are you saying that I smell?17:16
didrocksseb128: I would find that offensive as well17:17
tedgHeh17:17
tedgI'm saying you age nicely.17:17
didrocks:)17:17
seb128lol17:18
=== salem_ is now known as _salem
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== gatox is now known as gatox_lunch
=== gatox_lunch is now known as gatox
=== _salem is now known as salem_
=== salem_ is now known as _salem
=== jussi01 is now known as jussi
=== _thumper_ is now known as thumper
alo21hi all..22:19
alo21I read some ideas here: https://wiki.ubuntu.com/Unity/Lenses/Ideas22:19
alo21and I would to implement one of them. What should I do?22:20

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