didrockshey Mirv, how was your week-end?07:01
Mirvdidrocks: hello, very relaxing thank you07:39
didrocksnice! I hope the weather is better than what's I'm seeing here :)07:40
didrockshey sil2100! nice week-end as well?07:40
Mirvpretty good today, spring if finally arriving07:41
didrockslucky you :-)07:41
didrocksMirv: did you get some progressed on autopilot vs sdk package?07:41
Mirvdidrocks: yes, I managed to fix all the tests eventually, in addition to adding the separate autopilot package. waiting for my merge proposal to be accepted, after which I'll contact Francis to start running the tests.07:45
didrocksMirv: oh, you don't need to contact him, we can add them to our generic autopilot job for daily release :)07:46
didrocksMirv: awesome news! good job :)07:46
Mirvdidrocks: ah, ok. thanks.07:47
didrocksMirv: do you mind updating this spreadsheet please: https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0AuDk72Lpx8U5dFVVX1BOYm1qdUtyX2xUNmdwWlhTS0E#gid=007:47
didrocksMirv: with the command to run and so on :)07:47
didrocksI think you already did it :)07:47
didrocks(just double checking)07:47
Mirvdidrocks: yes, I did :)07:47
didrocksexcellent ;)07:48
didrocksMirv: do you have time to check with in the daily-build-next ppa, we have a lot of unity stack failures? (not the touch stack, only the unity one)07:50
Mirvdidrocks: not right away, but I can take a look in the afternoon07:51
didrocksthanks :)07:51
dednickSaviq: ping07:53
Saviqdednick, pong07:53
dednickSaviq: do you know if there's something wrong with the qmluitest jenkins build? keeps failing. http://s-jenkins:8080/job/unity-phablet-qmluitests/493/console07:53
dednickall i can see is that it can't connect to dbus, but dont know if that is the root of problem07:54
Saviqdednick, nah, the problem is "QXcbConnection: XCB error: 148 (Unknown), sequence: 148, resource id: 0, major code: 140 (Unknown), minor code: 20"07:54
Saviqdednick, unfortunately we don't have a solution for that yet07:55
Saviqdednick, we just restart the job and hope for the best...07:55
dednickSaviq: i c. ok07:55
Saviqmzanetti, anything to add ^?07:55
sil2100didrocks: hi! A very busy weekend sadly, but oh well! How about you?07:56
didrockssil2100: was uninteresting, stayed at home most of the time due to the bad weather07:56
sil2100Bad weather? Oh my, here in Poland we had sunny sunshine almost all the time07:57
didrockswaow, lucky, here since Friday, the weather isn't really nice…07:57
dednickSaviq: can i restart jobs? all i see in the "build again" page is "publish again". is that the same?07:57
Saviqdednick, you need to log in to s-jenkins07:58
dednickSaviq: nevermind. yeah, i realised i wasn't logged in07:58
didrockssil2100: how busy are you today? we are going to switch the new HUD after the finale touch quantal image is in (today normally)07:58
didrockssil2100: I wonder if meanwhile you can look at the touch-related build failures in the ppa07:58
dednickCimi: ping07:59
sil2100didrocks: ok, will look - in the daily-next PPA, yes?07:59
didrockssil2100: right! thanks :)07:59
Saviqtsdgeos, do you know of a way for automoc to play nice with headers kept separately from source files?07:59
tsdgeosSaviq: have never found myself in need of that08:00
tsdgeosso nope08:00
tsdgeosSaviq: what's the rationale for such a split?08:15
Saviqtsdgeos, better structure, really, but I'm not convinced, either08:17
tsdgeosto me it seems worse structure08:17
tsdgeoswon't be able to find a header by just doing s/.cpp/.h08:17
tsdgeosthat's always confused the hell out of me08:18
Saviqdednick, commit message missing https://code.launchpad.net/~nick-dedekind/unity/phablet-greeter-indicators/+merge/15733008:25
Saviqtsdgeos, yeah I actually agree, I'd just put actual includes in /include08:25
dednickSaviq: thanks. done08:25
mhr3dednick, using ff or chrome?08:26
dednickmhr3: chrome08:26
mhr3dednick, http://userscripts.org/scripts/show/139081 ;)08:26
dednickmhr3: normally.08:26
mhr3> Also adds a warning if there's no commit message specified for the merge proposal.08:26
Saviqtsdgeos, where only files that others might want to include, live08:27
dednickmhr3: cool. thanks :)08:27
tsdgeosSaviq: you mean for public vs private includes?08:28
Saviqtsdgeos, yeah08:28
tsdgeospersonally i like the .h/_p.h that Qt uses08:28
tsdgeosbut don't have any strong opinion either08:28
tsdgeosmzanetti: easy one https://code.launchpad.net/~aacid/unity/filtergrid_test_findChild/+merge/16003408:48
tsdgeosmzanetti: sadly we still need one of the __testing properties08:49
tsdgeossince behaviours are not part of children of a item08:49
tsdgeosso there's some stuff we can't get to even if we give an objectName :/08:49
tsdgeosmaybe in the C++ side we can?08:49
mzanettitsdgeos: same with Timers btw08:55
mzanettitsdgeos: would be awesome to have something where we could get them... but so far I couldn't come up with anything (didn't invest too much time tho)08:55
tsdgeostried using C++ QObject::children() ?08:56
mzanettitsdgeos: nope08:56
Cimidednick, hi nick09:24
tsdgeosmzanetti: might work09:32
tsdgeosback in a sec, my X froze and now it's doing weird stuff09:34
SaviqMacSlow, you might need to resubmit your renderer MP, LP seems to have gotten confused and has not included the latest changes (still shows conflicts)09:39
MacSlowSaviq, argl... hm... ok09:40
SaviqMacSlow, also, I'd like to get rid of the print()s09:40
SaviqMacSlow, I'd convert them into a signal clicked(string)09:41
SaviqMacSlow, so that you can then test that they got clicked (or not) in the tests09:41
SaviqMacSlow, you'll need a signal clicked(string) on both Notification and Notifications09:41
SaviqMacSlow, and in Notifications' delegate you'll forward that signal up09:42
SaviqMacSlow, and then in the tests use SignalSpy component to check if you can actually click on the actions09:43
Saviqmzanetti, can you please have a quick look through https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/15551209:43
mzanettiSaviq: ack09:43
Saviqmzanetti, and see if you come up with anything else that makes sense testing09:43
Saviqmzanetti, read up, too09:43
Saviqmzanetti, thanks09:43
MacSlowSaviq, ok09:44
sil2100didrocks: ok, so:09:48
sil2100didrocks: https://code.launchpad.net/~sil2100/indicator-messages/fix_ftbfs_deprecated_func/+merge/160038 <- for the indicator-messages FTBFS09:49
sil2100didrocks: I also have a branch for the gallery-app FTBFS but still waiting for a PPA test build09:49
didrockssil2100: excellent! thanks :)09:50
tsdgeosSaviq: how does https://code.launchpad.net/~aacid/unity/filtergrid_remove_itemY/+merge/160050 look for the itemY function removal?09:52
Saviqtsdgeos, we don't want to pass delegates around09:52
tsdgeosSaviq: why not?09:53
Saviqtsdgeos, they get destroyed when you're not looking ;)09:53
tsdgeossure, the only thing you have to do is not store them :D09:53
Saviqmzanetti, ^ ;)09:53
tsdgeosSaviq: well, i can pass up the "y" that's the only thing i need from the delegate09:54
tsdgeosbut note the carousel is already passing the delegate up09:55
tsdgeosmaybe not in the listview version, let me check09:55
tsdgeosnope, still there09:55
mzanettitsdgeos: in my experience using delegates inevitably leads to bad code... I do know that there are cases where its not harmful. But so far in my experience once you do that you start introducing work arounds in future revisions.09:56
Saviqtsdgeos, my thinking was to pass up a Qt.rect()09:56
mzanettibest example: notes-app09:56
Saviqtsdgeos, that should get parented to the Qt global object anyway09:56
Saviqtsdgeos, but then we need to make sure to delete it afterwards09:56
mzanettiwas a listView and they were using the currentItem outside of the listview. next MP changed it to be a Column + Repeater because it was crashing sometimes with the listview...09:57
Saviqtsdgeos, and would give us all we need (i.e. in People on phone the split is _after_ the clicked item, not before it)09:57
tsdgeosmzanetti: but that's like forbidding pointers because using them wrongly will make you crash :D09:57
tsdgeosSaviq: sure the rect is enough, but as said also the Y is enough09:58
mzanettitsdgeos: hmm.. I think its different... you don't realize its a pointer in QML while you do in C++.09:58
Saviqtsdgeos, sure, here y is enough09:58
tsdgeosmzanetti: you don't realize because you have no clue of QML (not you as you but you as general you)09:58
mzanettitsdgeos: yes... that might be true09:58
Saviqtsdgeos, in people y and height are needed09:58
tsdgeosSaviq: well, it's using peopleGrid.cellHeight so "not really"09:59
Saviqtsdgeos, right09:59
mzanettianyways, if possible to avoid using delegates I'd vote for that. I won't cut off your head if you do so tho...09:59
tsdgeosok, i'll pass up the Y then, no need to care about creating new objects and doesn't invalidate the delegates are evil "rule"10:03
dednickdoes anyone know why when i try load Shell.qml with qmlscene, it segfaults in the Unity plugin export? http://pastebin.ubuntu.com/5592114/10:09
dednickI expanded out the QT_MOC_EXPORT_PLUGIN def and it seems to be popping creating object on stack.10:11
* tsdgeos shouts at bazar again10:11
tsdgeossorry :D10:12
sil2100didrocks: is there someone already looking at the unity amd64 failure?10:12
tsdgeosdednick: no idea here10:12
didrockssil2100: I asked Mirv to have a look if he has time, if you have any, that would be appreciated. Did you finish with the powerpc failures? is it transient?10:12
tsdgeosmzanetti: autolanding 228 and 229 are of the same branch+rev?10:23
* mzanetti checks10:24
Saviqmzanetti, btw, any idea why the drop in line coverage http://s-jenkins:8080/job/unity-phablet-autolanding/ ?10:25
mzanettitsdgeos: indeed. no idea how that happened10:26
mzanettiSaviq: seems we added lots of new lines of code. biggest part caused by: obj-x86_64-linux-gnu.tests.mocks.Ubuntu.Application10:29
Saviqmzanetti, yeah, that's what I thought10:30
Saviqmzanetti, but then should we not ignore tests for coverage?10:30
mzanettiSaviq: yes. this is collected by the c++ coverage run.10:30
tsdgeosmzanetti: can you reapprove https://code.launchpad.net/~aacid/unity/filtergrid_test_findChild/+merge/160034 i needed to merge with trunk for it to pass10:30
mzanettiSaviq: would need to find out how to exclude the mock dirs10:30
Saviqmzanetti, k10:31
tsdgeosoh too late, autolanding already :D10:31
tsdgeosit'll fail now10:31
Mirvsil2100: feel free to start looking at that build failure. this afternoon looks to be becoming busier all the time10:32
sil2100didrocks: it seems to have been caused by the same little issue we had some time ago - a rebuild should help theoretically, but LP still didn't start rebuilding when I asked him to ;p10:33
sil2100Mirv: ok ;)10:33
MacSlowSaviq, "Unity.Test" needs a UnityTestQml... were is that meant to be coming from?10:33
sil2100What is this? It's still10:33
sil2100I mean, it's still "Start in 1 minute" since 30 minutes10:33
Mirvsil2100: I've seen that often10:34
didrockssil2100: yeah, in fact, it's "start in 1 minute after the next build finish"10:34
sil2100Ok, so that's confusing10:35
SaviqMacSlow, Unity.Test is in tests/qmltests/utils/10:35
SaviqMacSlow, along with the qml file10:35
didrocksSaviq: indeed :)10:35
didrockssil2100: ^10:35
sil2100But some of the timers are anyway broken - on some other LP package build I started I have 'Starts in 1 hour' for an hour already10:36
tsdgeosmzanetti: well, that UT thing is what Saviq used, i just copied it :D10:36
mzanettistill don't like it :P10:36
tsdgeosmzanetti: may as well change them all in one go10:36
tsdgeosi'm was jsut being consistent10:36
Saviqtsdgeos, mzanetti, if we want more explicit named import, we can, but I'm not sure I want "UnityTest.UnityTestCase"10:37
mzanettiSaviq: whats wrong with Unity.Test?10:37
Saviqmzanetti, it's polluting the global namespace if you don't do named imports10:38
Saviqmzanetti, and it's less obvious where it comes from10:38
Saviqmzanetti, actually we should probably s/UnityTestCase/TestCase/10:38
Saviqmzanetti, and use UT.TestCase or whatever10:38
mzanettiSaviq: well in that case we would need the named import, yes10:39
Saviqmzanetti, we want named imports to be used more, btw10:39
mzanettido we...10:39
SaviqI do10:39
mzanettiwell then...10:39
Saviqit's namespaces10:39
Saviqthey're useful10:39
Saviqmore so than "MyCustomNameToAvoidConflictsTestCase"10:40
mzanettiI know... they're useful - but annoying when overused10:40
mzanettifor example writing QML in Plasma is disgusting10:40
Saviqmzanetti, I'm good with changing the import name, but I'd rather keep the named import in10:49
Cimitsdgeos, you sure there's a conflict here? https://code.launchpad.net/~unity-team/unity/phablet.test_LensView_new/+merge/15964311:00
Cimitsdgeos, just merged trunk and seems fine11:00
tsdgeosCimi: it's what all the CI stuff complained about11:02
=== MacSlow is now known as MacSlow|lunch
tsdgeosCimi: is that test testing anything from listview?11:19
tsdgeosso what's the point? :D11:21
Cimitsdgeos, it's lensview11:23
Cimitsdgeos, not listview11:23
tsdgeosCimi: is that test testing anything from lensview?11:23
Cimitsdgeos, yes, searchquery11:23
Cimitsdgeos, I don't think there is else to test11:23
nic-doffaySaviq, could you take a look at this branch for me? https://code.launchpad.net/~nicolas-doffay/unity/infographics11:24
tsdgeosCimi: it searchquery is in lens11:24
tsdgeosand the lens you're using there is the fakeone11:24
tsdgeosonTriggered: lens.searchQuery = ""11:24
nic-doffayIn particular the Sequential Animation. The property action for the visibility doesn't work, I'm clueless as to why. http://bazaar.launchpad.net/~nicolas-doffay/unity/infographics/revision/563#Greeter/Dot.qml11:24
tsdgeosmay be worth testing11:24
tsdgeosyou you're not testing that either11:24
Cimitsdgeos, I'm testing that11:28
Cimitsdgeos, the timer gets triggered when lens changes11:28
Cimilook at the test11:28
tsdgeosi know11:28
tsdgeosbut the timer takes 2 secs11:28
tsdgeosand the tests does11:28
tsdgeoschange + compare11:28
tsdgeoshow does that work?11:28
tsdgeosit doesn't11:29
Cimimmm so I should to tryCompare11:29
tsdgeosthe thing is11:29
tsdgeosit's working because of something else11:29
Cimitsdgeos, on triggered changes11:29
Cimiit's istant11:29
tsdgeosCimi: i put a console.log("EEEE", lens.searchQuery) on that timer triggered11:29
tsdgeosgot nothing11:29
tsdgeoswhy would it be instant if the timer is 2 seconds?11:30
Cimitimer.restart doesn't trigger the timer?11:31
tsdgeosit starts the timer11:32
tsdgeosit will take 2 seconds to trigger11:32
Cimitsdgeos, I see11:33
tsdgeoswhat you want is something like this actually11:33
Cimitsdgeos, searchQuery is per lens11:33
tsdgeosbut this fails11:33
Cimitsdgeos, so if you change lens11:33
Cimitsdgeos, the new lens has empty searchQuery11:33
tsdgeosthen adding the tryCompare to the second one will exercise the timer probbly11:33
Cimibecause I previously set searchQuery for another lens11:33
CimiI should ideally go back to the initial lens where I set searchQuery and wait for the timer11:34
tsdgeosthat's what my paste does11:34
Cimijust saw your paste :)11:34
Cimitsdgeos, that? http://paste.ubuntu.com/5592300/11:38
Cimimight remove the first compare actually11:38
tsdgeosyeah, first compare may not make sense11:42
Cimiok pushed11:43
Saviqnic-doffay, yeah, looking11:46
tsdgeosCimi: do we need the "Component.onCompleted: lens = lenses.get(0)" ?11:47
Cimitsdgeos, not really if we have the init()11:47
tsdgeosCimi: kill it then :-)11:47
Cimitsdgeos, was just to set up the lensview11:47
Cimias it is used elsewhere11:48
Saviqdandrader, now there's a conflict in your close from dash branch11:48
tsdgeosCimi: but doesn't init do that already?11:49
Saviqnic-doffay, you need to merge trunk in your branch11:49
nic-doffayWill do now Saviq11:51
sil2100Ah ha\11:54
mzanettiSaviq: https://code.launchpad.net/~mzanetti/unity/phablet-unload-far-away-images/+merge/16008311:58
Saviqmzanetti, nice11:58
=== MacSlow|lunch is now known as MacSlow
=== dednick is now known as dednick|lunch
=== ritz__ is now known as ritz
mzanettiSaviq: greyback is currently working on fixing LVWPH, right? (when he's not on vacation)12:34
Saviqmzanetti, yes12:34
mzanettiSaviq: will it be changed so that only visible content is loaded?12:35
Saviqmzanetti, yes, that's the idea12:35
mzanettiok. thanks. I've commented on my own MP with some thoughts on this12:35
mzanettiSaviq: ^12:35
Saviqmzanetti, once we get rid of cacheBuffer there, we will unload the whole categories12:36
Saviqmzanetti, but then there's People :/12:36
Saviqmzanetti, where we have an unlimited list of entries12:37
Saviqin a ListView that really behaves like a Column + Repeater12:37
Saviqbecause it doesn't know if it's offscreen or not12:37
mzanettiSaviq: yeah... that's exactly what I would call the LVWPH issue12:38
=== seb128_ is now known as seb128
Saviqmzanetti, tsdgeos, dandrader I'd like you to have a look at https://code.launchpad.net/~saviq/+junk/interface-tests and say what's understood, what is not, do you see anything added there etc.12:39
dandraderSaviq, ok12:39
Saviqmzanetti, tsdgeos, dandrader, it's something of a mine field for the interface tests12:39
mzanettiSaviq: for things like test_type() I'd use _data() methods. If one fails we get better reports which type failed etc12:52
mzanettiSaviq: test_creation() could also test for !creation of uncreatableTypes?12:52
Saviqmzanetti, I don't think we want that, if they're creatable, nothing breaks12:53
mzanettiSaviq: ok12:53
Saviqmzanetti, same for writable12:53
SaviqI initially tested for !writable12:53
Saviqbut then thought it doesn't make sense12:53
dandraderSaviq, I didn't get tst_TestInterface.qml:17. Where that TestCase comes from?12:54
mzanettiSaviq: verifyProperty(): there could be properties that are CONSTANT and for thus don't have a NOTIFY signal. If this is intended to be a generic template for such tests it should reagard that12:55
mzanettidandrader: that's not a testcase but a helper function for other cases12:56
Saviqmzanetti, right12:57
Saviqdandrader, that TestCase comes from the Util module12:57
Saviqdandrader, (that's gonna be moved to the Unity.Test module when we deem it complete)12:58
Saviqdandrader, but for now this is just an example set12:58
dandraderSaviq, because I didn't see it being defined in the Util module itself12:59
Saviqdandrader, qmldir12:59
Saviqdandrader, it's a qml file12:59
mzanettioh... I seem to have another diff with different line numbers... sorry for the noise dandrader13:00
dandraderI see it now13:00
mzanettiSaviq: there could be a signalspy attached to verifyProperty() in case its writable13:04
Saviqmzanetti, yeah, I thought of that (and forgot)13:04
mzanettiSaviq: I think the rest looks good. can't think of anything else right now13:04
Saviqmzanetti, but then the actual signal might be delayed13:04
Saviqmzanetti, depending on the implementation13:04
mzanettiSaviq: hmm... it shouldn't13:05
Saviqright, it shouldn't indeed13:05
mzanettiSaviq: hmm. that could be the case indeed...13:05
mzanettiSaviq: but you could use a signalSpy.wait(). more than 5 secs delay is definitely not ok13:06
Saviqmzanetti, yeah probably13:06
mzanettiSaviq: I'm thinking of remote control apps, where a setVolume might need the RTT until it reflects that change13:06
Saviqtsdgeos, gotta run to the tax office, might not make it back for the standup, you're in charge13:07
Saviqmzanetti, it's tests, they won't be ran against an actual service13:07
dandraderSaviq, about naming: Int QtTest "verify" is used for checking that a boolean expression is true, whereas "compare" is used to compare that an actual value matches the expected one. I think we should keep that convention13:07
mzanettiSaviq: right... in that case... I'd add a spy13:08
dandraderso verify() is like an assert()13:08
tsdgeosSaviq: actaully i am in a meeting, not sure it'll make it either13:12
mzanettihehe... MacSlow can't make it either... it'll be a quick one today :)13:15
mzanettianyways, I'll take care of it13:15
dandraderSaviq, for the sake of readability I would consider splitting verifyProperty into two separate functions (it has too many arguments). One for checking the existence of the property and its type and another for trying to write to it.13:15
dandraderSaviq, or maybe having a read-only and read-and-write versions of it13:16
Cimidednick|lunch, planning on add anything here? https://code.launchpad.net/~unity-team/unity/phablet.fake-unity-plugin_categories/+merge/15962913:18
=== dednick|lunch is now known as dednick
dednickCimi: nope. it's done. Can you check your lensview branch against it?13:23
Cimiwill do13:23
Cimidednick, because it segfaults without13:23
dednickCimi: eh? what segfaults?13:24
Cimilensview tests...13:24
Cimidoesn't segfault on the pc13:25
Cimibut does on CI13:25
tsdgeosmzanetti: Cimi: guys won't make it to the standup, can you guys drive it?13:31
Cimitsdgeos, ok13:31
Cimidednick, nic-doffay  you guys joining the standup today?13:32
nic-doffayYeah one sec.13:32
dandraderdednick, as for the fake ApplicationManager issue when running with qmlscene. you could replace it with a very simple qml/js implementation13:45
dandraderdednick, This is done, if I'm not mistaken, in the Stage test13:45
dednickdandrader: since all the files reference ApplicationManager directly, it might be quite hard to do13:47
dednickdandrader: i dont actually know where the ApplicationManager is created tbh.13:47
dandraderdednick, or you could improve the fake ApplicationManager to make it work with qmlscene13:48
dandraderdednick, tests/mocks/Ubuntu/Application/ApplicationManager.cpp13:48
dednickdednick: yup. thats the idea :)13:49
dednicklol. talking to myseld13:49
dandraderdednick, the code is making some assumptions that seem to not hold when run from qmlscene13:49
dednickdandrader: thats the idea :) got it working, but dont know if it's a good solution or not13:49
dandraderdednick, it should be easy to spot those places13:50
dednickdandrader: thanks.13:50
dandraderin the code13:50
ChrisTownsendmhr3: Hi13:50
dandraderdednick, I actually added comments abou tit13:50
dandraderabout it13:50
dednickdandrader: yup, i've seen them :)13:50
dandradermzanetti, it's in lp:~dandrader/unity/phablet_close_apps_from_dash_WIP13:54
dandradermzanetti, UnityTestCase::onCalendarLongPressed() is not getting called on the second "press and hold"13:54
Saviqmzanetti, thanks13:55
Saviqdandrader, re verify vs. compare, which function did you have in mind?13:55
dandraderSaviq, all of them,13:56
Saviqdandrader, I'm not sure, it's not just about comparing two values13:57
dandraderSaviq, I mean oll of the verifyFoo(..., actual, expected)13:57
dandraderSaviq, hmm, right13:58
=== dandrader is now known as dandrader|brb
tsdgeosmzanetti: Cimi: standup still running?14:02
=== mmrazik is now known as mmrazik|otp
mzanettitsdgeos: no14:03
didrockssil2100: seems the build relaunch worked \o/14:08
didrockssil2100: camera-app still have the python dep missing, isn't it?14:08
sil2100didrocks: the thing is, it has the python dep in it :|14:09
didrockssil2100: do you reproduce in a pbuilder?14:09
sil2100didrocks: and python2.7 along with stdlib is installed correctly - and I was unable to reproduce it locally here14:10
sil2100Also, the ap CMakeLists.txt code is the same as in gallery-app and phone-app, and it works fine there14:10
tsdgeosCimi: why did you make https://code.launchpad.net/~unity-team/unity/phablet.test_LensView_new/+merge/160108 depend on the categories one?14:10
sil2100Doing some testing on PPA's but it's hm taking a while14:10
didrockssil2100: you didn't enable a pbuilder on your machine, do you need help for that?14:11
sil2100didrocks: I use pbuild14:11
sil2100I think that's almost the same, just with some wrappers around it14:12
didrockssil2100: ok ;) you did notice some other errors in term of source package name in the media stack as well?14:12
Cimitsdgeos, because I thought the segfault was due to the missing categories14:13
Cimitsdgeos, in reality is on the opengl14:13
Cimitsdgeos, but the categories is still ready for review so I suppose we can review that first so it doesn't have the warning too14:14
sil2100didrocks: those, ah! Will push a merge now ;)14:15
=== dandrader|brb is now known as dandrader
tsdgeosCimi: you reviewing https://code.launchpad.net/~unity-team/unity/phablet.fake-unity-plugin_categories/+merge/159629 or want me to take a shot at it?14:15
didrocksthanks sil2100 ;)14:15
=== mmrazik|otp is now known as mmrazik
Saviqdandrader, re: simplyfing verifyType14:18
tsdgeosCimi: oh wait you proposed it, so you can't review it :D i'll have a look14:19
Saviqdandrader, I'm thinking of creating a custom class that you would create and pass the object and its readable name to14:19
Saviqdandrader, and then call verifyProperty() with the reduced set of args14:20
dandraderSaviq, sounds promising14:23
Saviqnic-doffay, you might want to have a look at lp:~saviq/+junk/interface-tests14:25
Saviqnic-doffay, it's not complete and I will have some changes to it, but should be a good start14:25
nic-doffayThanks Saviq14:26
mzanettinic-doffay: so...back from the QA meeting. hit me14:29
Cimiwhen genericLensView is used?14:29
Cimiapparently it's a property in DashContent14:29
Cimia string14:30
Cimioh used as delegate nevermind14:31
Ciminot used currently14:31
nic-doffaymzanetti, check this out quickly http://bazaar.launchpad.net/~nicolas-doffay/unity/infographics_transitions/revision/617#Greeter/Dot.qml14:31
nic-doffayPropertyAction on line 38.14:31
nic-doffayIt's supposed to do something very simple.14:31
nic-doffayHere's the branch link if you'd like. https://code.launchpad.net/~nicolas-doffay/unity/infographics_transitions14:32
mzanettinic-doffay: just to be clear... line 38 does not have a PropertyAction. but I suppose you mean line 39, right?14:33
Saviqnic-doffay, Pause, Action 1, Action 2 << that means Action 1 simply won't have any effect, 'cause it's overwritten by Action 2 straight away14:34
mzanettiright... yeah. Saviq is right14:35
mzanettiof course he is14:35
nic-doffaySo a pause or an animation is best then?14:35
mzanettiSaviq: no... actually, its different properties14:35
Saviqis it?14:36
sil2100didrocks: https://code.launchpad.net/~sil2100/qtubuntu-camera/rename_source/+merge/16012514:36
sil2100didrocks: https://code.launchpad.net/~sil2100/qtubuntu-cameraplugin-fake/rename_source/+merge/16012814:36
Saviqmzanetti, ah... /me hides14:36
Saviqmzanetti, in mtg, can't really focus14:37
sil2100didrocks: one of them has also adding the bootstrapping commit14:37
mzanettino worries14:37
mzanettinic-doffay: how to test this?14:37
mzanetticlicking the infographic like crazy doesn't seem to do anything14:38
nic-doffayYou mean nothing is visible mzanetti ?14:39
mzanettinic-doffay: I think this is not used in the Greeter yet, is it? Anyways, I've run Infographics.qml in qmlscene and its just black14:40
nic-doffaymzanetti, that's why I was after some assistance because the PropertyAction isn't enabling the visibility.14:41
mzanettinic-doffay: right... got it14:41
nic-doffaymzanetti, would the best be to use an animation or a pause between the action and the next animation?14:41
mzanettinic-doffay: not really sure I understand the question. but I've a solution to your first issue14:42
mzanettinic-doffay: sprite.visible == false => all its childs will be invisible too14:43
mzanettinic-doffay: I've sett visible: true and now clicking somewhere makes the dots start changing appearance14:44
nic-doffaymzanetti, yes.14:44
mzanettinic-doffay: is that what it should be or am I missing something?14:44
nic-doffaymzanetti, I don't think I explained this well enough from the get go.14:44
nic-doffayWhat needs to happen is the following:14:44
nic-doffayEverything is invisible.14:45
dandradermzanetti, did you get to try my branch? I'm still clueless14:45
nic-doffayOpen a click (or something else in future) the dot in question becomes visible then changes it's state.14:45
mzanettidandrader: I branched and compiled already...14:45
=== alan_g is now known as alan_g|afk
mzanettinic-doffay: so what you want is to change sprite.visible to true upon the click?14:48
didrockssil2100: thanks! :)14:49
nic-doffaymzanetti, or the dot. I figured it wouldn't make a difference since the sprite is a child of the dot.14:49
mzanettinic-doffay: well, it does make a difference in a sense that changing a parents visiblity changes ALL the childs too14:50
mzanettinic-doffay: so if you only want one (or some) dot to become visible, you can't use the sprite14:50
nic-doffaymzanetti, I know. In this case it doesn't matter.14:51
nic-doffayNothing should be visible.14:51
tsdgeosSaviq: you doing https://code.launchpad.net/~aacid/unity/filtergrid_remove_itemY/+merge/160050 or want me to find someone else?14:51
Saviqtsdgeos, in mtg, try someone else :)14:52
mzanettinic-doffay: if you move the visble: false from the sprite up to the dot. is it then what it shoul be?14:52
tsdgeosSaviq: okidoki14:52
nic-doffaymzanetti, doh! I just noticed that.14:53
tsdgeosmzanetti: since you oposed to the one using the delegates can you do https://code.launchpad.net/~aacid/unity/filtergrid_remove_itemY/+merge/160050 ?14:53
mzanettitsdgeos: hehe, sure.14:53
mzanettibut need to checkout dandrader's issue first14:53
nic-doffaymzanetti, sorted, thanks for the extra pair of eyes :)14:53
mzanettinic-doffay: ok, great14:53
mzanettidandrader: how do you launch this?14:54
mzanettiI always get Ubuntu.Application 0.1 is not installed14:54
dandradermzanetti,  make testRunningApplicationsGrid14:54
mzanettidandrader: in qmlscene14:55
dandradermzanetti,  You have to pass the built tests/mocks dir as an import path14:55
dandraderwith -I14:56
* mzanetti is a bit lost with the too many plugins dirs in tests14:56
mzanettidid I suggest this?14:56
* mzanetti beats himself14:56
dandradermzanetti, ask Saviq  :)14:56
kgunnSaviq: we ok to switch https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/155512 to "Needs Review" ??14:57
Cimimzanetti, what is good to be tested for genericlensview?14:58
Saviqkgunn, yeah14:58
kgunnMacSlow: ^14:58
Saviqkgunn, after we have the action tests we can get it in14:58
mzanettiCimi: I'll come back to you in a bit14:58
Cimiok thx14:58
=== alan_g|afk is now known as alan_g
kgunnMacSlow: might want to poke loicm on the alpha adoption for the icon15:00
kgunnMacSlow: maybe just a missing capability we might have to add ?15:00
MacSlowkgunn, the question is... is it just a very special use-case for notifications (-> use priv. Component) or could other apps use it too (-> extend UbuntuShape)15:02
mzanettidandrader: hmm... I might be wrong, but I think this line 144 should be "tryCompareFunction(checkSwitchToActivationModeAfterLongPress, false)"15:02
MacSlowkgunn, I'll email loicm about it15:02
mzanettidandrader: I can't see any problem about the longpress not happening. it works manually and in the test I too get the debug print "test_enterTerminationMode() long pressed tile!!!"15:03
dandradermzanetti, on, it should be true. the problem is that onCalendarLongPressed() is not getting called on the second long press15:03
dandradermzanetti, the long press happens twice, yes, but on the second time UnityTestCase::onCalendarLongPressed() doens't get called15:04
dandradermzanetti, and I have no idea why15:04
mzanettidandrader: so the issue is that this "calendarTile.onPressAndHold.connect(onCalendarLongPressed)" stops working after the first time it gets fired, right?15:05
dandradermzanetti, yes15:05
dandradermzanetti, I was using a signalspy before and switched to this approach thinking that the issue might have been in the SignalSpy itself15:05
kgunnMacSlow: exactly, was thinking...it might be a common desired effect15:05
dandradermzanetti, but both approaches show the same problem15:05
kgunnworth asking15:06
mzanettidandrader: ok15:06
dandradermzanetti, also, if I make this connection only before the second long-press, it still doesn't get called15:06
MacSlowkgunn, visual polish-wise I'm +1 for extending UbuntuShapre there15:06
Cimimzanetti, I only tested isCurrent resetting the searchQuery   lp:~unity-team/unity/phablet.test_GenericLensView15:07
=== _salem is now known as salem_
Saviqkgunn, MacSlow yeah, I wanted to mention that it should be UbuntuShape that "disables" itself if the image has alpha15:11
SaviqMacSlow, can you pick that up with the SDK guys - file a bug against ubuntu-ui-toolkit15:11
MacSlowSaviq, just CC'ing you and kgunn on the email I'm just writing15:11
mzanettidandrader: first click: long pressed tile!!! RunningApplicationTile_QMLTYPE_7(0x113b470, "runningAppTile Calendar")15:12
mzanettidandrader: second click: long pressed tile!!! RunningApplicationTile_QMLTYPE_7(0x10f9cf0, "runningAppTile Phone")15:12
mzanettidandrader: could that be the issue? seems the second click goes to the Phone tile15:12
dandraderah, stupid dandrader! nothing like a second pair of eyes to spot what the code author overlooked!!15:13
dandradermzanetti, thanks!15:13
mzanettidandrader: no problem. you're welcome15:13
Saviqmzanetti, dandrader, types = [plugins, modules, imports]; we need $types, tests for $types, $types for tests, tests for $types for tests15:13
Saviqmzanetti, dandrader, and then differentiate between mocks and utils15:13
mzanettiSaviq: yeah... I guess it does make sense15:14
Saviqmzanetti, dandrader, I'm good with simplifying the tree, but I can't think of a good way15:15
tsdgeosmzanetti: is jenkins syncinc broken or just slow? i.e. i just got an email pointing to https://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/710/console but it 404s15:15
mzanettiSaviq: need to use it for a day to tell if its good or not15:15
Saviqtsdgeos, I _think_ it's slow15:15
mzanettitsdgeos: I think Saviq has seen this last week too. in that case it was slowness15:16
Saviqbut that's new anyway15:16
mzanettiSaviq: tsdgeos: ok. I'll bring it up in QA15:16
Saviqmzanetti, thanks15:16
mzanettii.e. when mmrazik comes back online :D15:16
dandraderSaviq, well, I don't have any suggestions at the moment myself15:16
mhall119didrocks: is the pre-validation PPA still being updated?15:16
mhall119because after a reboot, I no longer have Unity :(15:17
kgunnnic-doffay: ping15:17
nic-doffaykgunn, what's up15:17
didrocksmhall119: remember I told to not using the pre-validation ppa but the certified one? :)15:18
didrocksmhall119: you can end up in a partial upgrade situation in this one, of with a bug that prevent unity starting15:18
mhall119no, I don't recall that15:18
mhall119so do I need to purge that ppa? and lose all my wonderful smart scopes?15:19
didrocksmhall119: you should use the certified ppa :)15:20
mhall119that sounds boring15:20
didrocksboring == less risky? :)15:20
didrockshowever, you need to install  ubuntu-unity-experimental-certified15:20
didrocksfrom it15:20
mhall119that's a package?15:20
didrocksto force to even downgrade some packages to always have things from that ppa15:20
didrocksyep :)15:20
didrocksyou won't live on edge, but you will be safe :)15:21
mhall119didrocks: is it worth it?  Or should I just drop back to default Unity until I can switch to Unity Next?15:21
mzanettiCimi: your turn.15:24
Cimimzanetti, :D15:24
mhr3mhall119, did noone tell you to not do dist-upgrades when they want to remove unity? :)15:24
mhr3mhall119, but the ppa is clean atm, not sure why you can't run it15:25
mzanettiCimi: first of all, it doesn't really add much except the thing you've tested already. so that's ok15:26
mzanettiCimi: I'm thinking if it would be possible to run LVWPH tests on it15:26
mhall119mhr3: I have unity installed, and running it from the command line doesn't show any errors, it just doesn't show up15:27
mzanettiCimi: just the exact same tests, but as input use the GenericLensView instead of LVWPH directly15:27
mzanettiCimi: and/or LensView tests15:27
mhr3mhall119, tried restarting? maybe driver bug15:28
mhr3weird then15:28
Cimimzanetti, mmm15:28
Cimimzanetti, gerry was working on LVWPH No?15:29
mzanettiCimi: yeah. its merged already15:29
mzanettiCimi: in a first step I would just use that file and exchange the input component with the GenericLensView15:29
mzanettiCimi: if that should work we can think of a clever way so we don't need to copy/paste all the tests15:29
mzanettiCimi: if it does not work I think there's not much more to test in there...15:30
Cimimzanetti, why we need to test lvwph?15:32
Cimiit's already tested15:32
tsdgeosCimi: what does "we can't import the original unity plugin" mean?15:33
mzanettiCimi: yeah... just thinking: The genericLensView is basically a LVWPH. it extends it just a little bit. but in that extension it could also break things. So if we could re-use just the same tests as for the LVWPH to make sure it still works in GenericLensView it would be awesome15:33
Cimitsdgeos, categories are in unity plugin15:33
tsdgeossure, i didn't say you use the unity plugin15:34
tsdgeosi did say why you're not using the categories code15:34
Cimitsdgeos, well it uses dee and other things...15:34
Cimitsdgeos, we can have something simpler15:34
tsdgeosbut it's not simpler15:34
=== racarr_ is now known as racarr
tsdgeosyour doing lots of code15:34
kgunnmzanetti: ping15:35
mzanettikgunn: hey15:35
tsdgeosyou have your own ::Data implementation15:35
tsdgeosCimi: who's going to keep that in sync with the real one?15:35
Cimitsdgeos, just the api, no?15:36
Cimitsdgeos, dednick wrote that code though, he know better then me :)15:36
tsdgeosdednick: why you not using the real categories code?15:36
dednicktsdgeos: because it's dee.15:36
tsdgeosdednick: so?15:37
dednickwe'd have to model it anyway15:37
dednickand keep it in sync15:37
=== dandrader is now known as dandrader|lunch
tsdgeosdednick: well, one thing is creating a fake dee model15:37
tsdgeosthe other is building a totally different categories code15:37
dednicktsdgeos: what's creating a fake dee model?15:37
mhall119didrocks: mhr3: can not open /usr/lib/compiz/libcore.so15:39
tsdgeosdednick: something like what i do in hud_client_query_get_results_model in ./tests/qmltests/plugins/HudClient/fake_libhud_client.cpp15:39
tsdgeosyou can always create a local dee model15:39
tsdgeosand use that in Categories15:39
tsdgeoswithout reimplementing the whole lot of the Categories class15:40
dednicktsdgeos: yeah, but then you're testing the categories, not the qml.15:40
mhr3mhall119, we don't even have compiz in the ppa, the distro one is used15:40
tsdgeosdednick: why are you not testing the qml if you do that?15:41
dednicktsdgeos: i15:41
dednicktsdgeos: i guess you're right. i remember thinking about doing that originally, but must have forgot about it.15:42
dednickor found a reason for not15:43
tsdgeosi'd really like if you could to try15:43
tsdgeosthe less glue cpp code we have the better15:43
Cimimzanetti, some works, I'm working on it15:44
nic-doffaymzanetti, another quick probably obvious question for you. I've been trying to centre a child using it's parent's anchors like this: anchors.verticalCenter: parent.Center15:45
nic-doffayIt doesn't affect a change though.15:46
Cimimmm no, lots of fail15:46
mzanettianchors.verticalCenter: parent.verticalCenter15:46
mzanettinic-doffay: ^15:46
nic-doffaymzanetti, sorted :)15:46
mzanettiCimi:  ok... just leave it then15:46
mzanettiCimi: its a nice theoretical idea but I am well aware in practice this can be tricky15:47
sil2100didrocks: hm, did you or anyone else redeploy the phone stack by any chance ;)?15:47
Cimimzanetti, I copied the unitytestcase over15:47
Cimifrom lvwph15:47
sil2100didrocks: since it seems the old one is still being build (failures related extra packages)15:47
Cimimzanetti, and added             property var pageHeader: findChild(genericLensView, "pageHeader")15:47
Cimi            property var listView: findChild(genericLensView, "listView")15:47
mzanettiCimi: and then it starts working?15:48
Ciminot really, some pass some don't15:48
ChrisTownsendmhr3: Hi, I have a question about your review of https://code.launchpad.net/~townsend/libunity/fix-search-hint-localization/+merge/15642015:48
mzanettiCimi: ok... just try a little bit to find out why they fail (e.g. its just too different to be tested in the same way vs. oh, its real bugs found already)15:49
didrockssil2100: it should be, let me try again :)15:49
mzanettiCimi: but you don't have to make it happen by any means15:49
mhr3ChrisTownsend, ah thx, see you need to ping people :)15:50
ChrisTownsendmhr3: Yep:)  I had actually forgotten about that MP, hence, no ping.15:51
Cimimzanetti, might be related to the different model used...15:54
mhall119mhr3: /usr/lib/compiz/libcore.so doesn't exist on my filesystem, and apt-file isn't finding it in any package either15:54
mzanettiCimi: right.. yeah the input data would need to be the same, yes15:54
mhr3mhall119, sounds like you broke your config then?15:55
Cimimzanetti, that won't work15:56
didrocksmhall119: indeed, there is no libcore.so… what did you install?15:56
Cimimzanetti, because GenericLensView uses the categories model15:56
mzanettiCimi: ah ok...15:56
didrocksmhall119: the core plugin is /usr/lib/libcompiz_core.so normally15:57
mzanettiCimi: ok then... I'd say its fine what you already have then15:57
mhall119didrocks: I just ppa-purged the prevalidation PPA, and tried to run Unity15:57
didrocksmhall119: can you pastebin a little bit more of your .xsession-errors? or what you see exactly once starting Unity?15:58
kgunntsdgeos: ping15:59
tsdgeoskgunn: hi15:59
mhall119didrocks: in a call atm16:07
mhall119running gnome-shell16:07
mhall119but I'll get you that when I'm done16:07
=== jasoncwarner__ is now known as jasoncwarner
ChrisTownsendmhr3: I assume you see my question in that MP, right?16:08
mhr3ChrisTownsend, yea, will re in a sec, discussing other stuff now16:08
ChrisTownsendmhr3: Ok, whenever you have a moment.  Just wanted to make sure we were on the same page.  Thanks!16:09
Saviqdandrader|lunch, I've updated lp:~saviq/+junk/interface-tests16:37
sil2100didrocks: https://code.launchpad.net/~sil2100/camera-app/modify_python_dep/+merge/160154 <- btw. I think this would be the fix for this issue16:41
smspillazdidrocks: mzanetti: libcompiz_core isn't hte core plugin16:42
smspillazwell ... it is ... kinda. Its just not dlopened )_16:42
mzanettismspillaz: huh?16:42
smspillazusually when compiz complains of a missing core plugin its trying to open some other plugin that has a mismatched core abi16:42
sil2100I checked in my local PPA and it helps, at least it seems to16:42
mzanettismspillaz: you sure you meant me?16:42
smspillazmzanetti: nope, I meant mhall119 :)16:43
didrockssmspillaz: ok, makes sense, I think you need python-minimal16:43
smspillaznow I'm confused16:43
didrockssil2100: ^16:43
didrocksstop talking people, too many s! :p16:43
=== smspillaz is now known as notsmspillaz
notsmspillazdoes this help ?16:43
didrockslet's do a criss-crossed talk ;)16:44
didrocksthanks notsmspillaz ;)16:44
=== notsmspillaz is now known as smspillaz
smspillazmhall119: in any case it sounds like you have an abi break somewhere16:44
tedgmpt, So I'm looking at networking, and I'm curious if the icon shouldn't match the precise percentage.  i.e < 10 should show up as none, but >75 should show up as full.16:45
tedgmpt, Perhaps we should be a bit optimistic on the icon?16:45
tedg"Ubuntu is so great, it gets strong wireless connections everywhere!"16:46
=== dandrader|lunch is now known as dandrader
Saviqmzanetti, lp:~saviq/+junk/interface-tests updated for you, too16:48
Saviqmzanetti, the signalspy is TODO still, though - tomorrow16:48
Saviqmzanetti, I might make _singletons and _roles into _data()-driven, too, tomorrow16:49
Saviqnot yet sure whether I want to16:49
mzanettiSaviq: ok... can I move this to tomorrow?16:50
Saviqmzanetti, yeah of course16:51
mzanettiSaviq: I still have two reviews in the queue16:51
Saviqmzanetti, you didn't think I meant "I can't be bothered to do the remaining changes today after my EOD, but you go on, review it NOW!", did you16:51
mzanettihehe... not really16:52
sil2100didrocks: ping! https://code.launchpad.net/~sil2100/webbrowser-app/bootstrap/+merge/16015716:53
mhall119didrocks: mhr3: xsession-errors: http://ubuntuone.com/1C5UYgKDI0UtHynnOOFdrK16:53
sil2100didrocks: since the Apps stack script was segfaulting because of lack of bootstrapping it seems ;p16:53
mhall119didrocks: mhr3: unity -vvvv output: http://ubuntuone.com/0b239apKA7DuivwjLfP48016:53
didrockssil2100: the version should be 0.20-0ubuntu116:54
tedglarsu, charles, is any indicator using on/off switches and GMenu yet?16:54
charlestedg: no, I don't think so16:54
mhr3mhall119, reset your compiz plugin list?16:55
tedgHmm, okay.16:55
mhall119mhr3: how?16:55
sil2100didrocks: pushed16:55
mhr3mhall119, dconf16:55
mhall119mhr3: got a handy-dandy command string I can copy/paste?16:55
mhr3i dont know the exact key sorry16:56
smspillazmhall119: compiz (core) - Info: Unity is not supported by your hardware. Enabling software rendering instead (slow).16:59
smspillazcompiz (core) - Info: Unity is not supported by your hardware. Enabling software rendering instead (slow).16:59
smspillazdo a dist upgrade lately ?16:59
smspillaz(don't worry about the "failed to load plugin ... blah blah blah" errors on --debug, its just complaining every time it tries to open something in ~/.compiz-1/plugins and nothing is there16:59
smspillazmhall119: though curiously, it says its supported later17:01
smspillazmhall119: in any case, looks like you're not loading unity shell. Have  look at the value for gsettings get org.compiz.core:/org/compiz/profiles/unity/core/ active-plugins17:02
=== alan_g_ is now known as alan_g
MarlincDid Unity really improve that much since 12.10 performance wise? How can it be so fast now? Because it the same codebase as Unity Touch and that that is optimized for phones now?17:03
mhall119smspillaz: yeah, I did a dist-upgrade17:03
mhall119it's supported hardware, decent but boring Intel GPU17:03
smspillazMarlinc: its the same codebase as 12.1017:03
smspillazmhall119: just minor changes here and there17:03
dednickCimi: i've removed the categories in favour of dee models17:03
smspillazMarlinc: just minor changes here and there17:04
MarlincSo it really is faster?17:04
smspillazthere are too many "d"'s "m"'s and "s"'s in this channel17:04
smspillazMarlinc: *shrug* see for yourself17:04
bschaefersmspillaz, haha17:04
MarlincI'll wait till it is released but I hear the development is very stable17:04
mhall119smspillaz: should 'core' be in the listof active-plugins?17:05
bschaeferI found someone who has like 4-5 prefix as someone elses name...i kept pinging the wrong person17:05
smspillazmhall119: yes17:05
Cimidednick, you sure it works?17:05
smspillazmhall119: though it should be more than just that17:07
nic-doffaymzanetti, another question. This gives me the results I expect, but it's giving an error on compile.17:07
nic-doffayproperty real value: parent.x + parent.horizontalCenter17:08
smspillazmhall119: ah, I knew I had typoed it17:09
smspillazgsettings get org.compiz.core:/org/compiz/profiles/unity/plugins/core/ active-plugins17:09
smspillaz(note the extra /plugins/)17:09
mhall119smspillaz: I used dconf-editor to reset it to default, still isn't working though17:10
smspillazmhall119: resetting it to the default won't work the way you expect17:10
mhall119smspillaz: ['core', 'ccp', 'composite', 'opengl', 'copytex', 'compiztoolbox', 'decor', 'imgpng', 'place', 'gnomecompat', 'snap', 'grid', 'resize', 'regex', 'animation', 'expo', 'vpswitch', 'move', 'mousepoll', 'session', 'workarounds', 'wall', 'ezoom', 'fade', 'scale']17:10
mhall119that's what it currenly has17:11
smspillazmhall119: append 'unityshell' to the end17:11
mzanettinic-doffay: what error message?17:11
Saviqnic-doffay, horizontalCenter isn't a real value17:11
Saviqnic-doffay, it's an AnchorLine17:11
nic-doffaySaviq, the thing is it does what I thought it would do.17:12
nic-doffayBut I'm getting this error: Unable to assign QString to double (obviously due to what you said)17:12
mzanettiwhich is the weird thing...17:12
mzanettiQString... even better :D17:12
Saviqnic-doffay, but it shouldn't be used like that ;)17:12
Saviqmzanetti, yeah, JS17:12
nic-doffaySaviq, what should I use instead?17:12
mhall119smspillaz: that did it!  I could kiss you17:12
Saviqnic-doffay, width / 217:12
smspillazmhall119: I feel like that would be a bad idea17:12
nic-doffaySaviq, I tried that, assuming it would give me the same result.17:13
nic-doffayIt didn't.17:13
Saviqnic-doffay, so it sounds like the approach you're taking now indeed is not giving the right result17:13
Saviqnic-doffay, what's your use case?17:14
nic-doffaySaviq, it's a bit complicated.17:14
Saviqnic-doffay, ok, regardless - if you're looking for a center of something, most often anchors is the way to go17:15
Saviqnic-doffay, but also remember x and y are local values in a hierarchy of objects17:16
Saviqnic-doffay, which means you might need mapToItem / mapFromItem, depending on what you're doing17:16
nic-doffayI'm assuming that converts x & y values from node spaces.17:17
nic-doffayBut that's not really what I need.17:17
Saviqnic-doffay, if you expected horizontalCenter to be half of width, then your equation should be: parent.x + (parent.width / 2)17:18
* Saviq never remembers which is horizontal and vertical in QML...17:19
nic-doffaySaviq, yeah that's obvious, but like I said, I attempted that.17:19
nic-doffayI'll have to dig deeper, this doesn't seem like a simple issue to solve.17:19
Saviqnic-doffay, « onValueChanged: console.debug(value) » to print out the value it gets17:19
nic-doffayCool thanks Saviq that will help a lot.17:20
dandraderSaviq,  lp:~saviq/+junk/interface-tests is looking good.17:24
Saviqdandrader, I didn't want for Verifier to subclass TestCase initially... but then nothing was left in there and having a separate Verifier type would just mean you'd have to pass references to it here and there17:28
dednickCimi: yup, all working17:28
dednickjust confirmed all Dash/DashContent & LensView tests working. Plus there is a test scene in Unity/categories which gives a view on all the categories of a lens.17:29
dednicks/categories/category results17:29
Saviqdandrader, so I'll leave it like that for now and if we decide it's bad we'll refactor17:30
=== boiko_ is now known as boiko
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
mterryfginther, where is the config for the generic-mediumtests autopilot test stored?  It seems to not have ppa:ubuntu-unity/daily-build-next in its sources.list, but it should18:40
fginthermterry, It's just a jenkins job. The ppa hook scripts are passed in from the parent job18:44
dobeyhey all, are there instructions for how to run unity-next on ubuntu 13.04 anywhere? (not how to flash to a phone, but how to run stuff locally)18:44
fginthermterry, so it's the upstream job that would need to be configured to use that ppa18:44
mterryfginther, from the CI job then?  OK.  You were working on adding that PPA to the jobs, right?  Any luck?18:45
mterryfginther, also, still getting some 404 errors, like with: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/584/console18:46
fginthermterry, yes that work now, but I had only deployed it for a few projects so far.18:46
fginthermterry, do you know the -ci job? i can be sure it's updated18:47
mterryfginther,  http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/708/18:47
mterryfginther, oh wait18:47
mterryfginther, for the PPA issue?  let me get some18:47
fginthermterry, hmm. it may just be easier to re-deploy all the stacks?18:48
mterryfginther, for the PPA issue, I'm looking at the various -app packages like phone-app, gallery-app etc18:48
mterryfginther, for the 404 issue, http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/708/ is the -ci job, from this merge: https://code.launchpad.net/~mterry/unity/phablet-greeter-lightdm/+merge/15984618:48
fginthermterry, ack. I'll make sure those are deployed and then update the other stacks18:49
fginthermterry, I know that jenkins.qa.ubuntu.com is being excessively slow to publish results. I've tried to find out why, but no luck yet18:54
=== seb128_ is now known as seb128
mhr3andyrock, did you restart all the scopes after installing my branch?22:21
andyrockmhr3, yeah i did logout/login too...22:22
andyrockdoes it work there?22:22
andyrocklet me try again22:23
andyrockmhr3, nope can't make it work22:34
andyrocki'm doing something wrong22:34
andyrockanyway it's late in Europe ;)22:34

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