/srv/irclogs.ubuntu.com/2014/05/22/#ubuntu-unity.txt

=== irsol_ is now known as irsol
=== duflu_ is now known as duflu
=== maxb_ is now known as maxb
tsdgeosSaviq: ok so tests are now run again, and obviously they don't pass again :D07:38
Saviqtsdgeos, that explains things ;)07:39
tsdgeosSaviq: fginther: and it seems we probably want to increase the 90 min limit?07:39
tsdgeoshttps://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/59/consoleFull07:39
Saviqtsdgeos, probably, yeah07:39
fginthertsdgeos, Saviq, I've bumped it to 180 minutes and stated another test07:43
Saviqfginther, thanks!07:43
tsdgeospaulliu: you working on fixing the test errors regarding zoom stuff at https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/59/consoleFull ?07:47
paulliutsdgeos: yes07:59
tsdgeoscool :)07:59
paulliutsdgeos: Can you review it?08:00
paulliutsdgeos: oh, sorry, I saw your comments.08:01
paulliutsdgeos: I'll fix it.08:01
CimiSaviq, pete-woods ping08:03
pete-woodsCimi: hi08:03
Cimimorning08:03
SaviqCimi, wassup?08:03
CimiSaviq, I was chatting yesterday with pete08:04
CimiSaviq, basically with the current file infographics I only need to use a crossfadeImage08:04
CimiSaviq, so we were wondering if we really need a model or not08:05
pete-woodsCimi, Saviq: http://paste.ubuntu.com/7500579/08:05
SaviqCimi, it's safer future-wise08:05
Cimimzanetti, if you want to join us again08:05
pete-woodscould have that API?08:05
pete-woodsit has the advantage that I know which infographic you looked at last, so I can save it08:05
Saviqpete-woods, I don't think the backend should have a notion of "current infographic"08:05
pete-woods(a requirement from design)08:05
tsdgeosMirv: can you update/rebuild/something libunity-scopes-dev in the beta2 ppa?08:06
CimiSaviq, issue with the model is that I need to use repeater or listview, which is not efficient on UI side08:06
pete-woodsSaviq: it's not technically the backend, it's just a QML plugin that talks to the backend08:06
pete-woods"talks" = not really talks08:06
Saviqpete-woods, sure, but then if someone decides (like it was planned before) that we'll be swiping between infographics again08:06
pete-woodsSaviq: okay, sure that's fair08:07
Saviqpete-woods, the API breaks08:07
pete-woodsSaviq: obviously I would really prefer to do nothing :)08:07
pete-woodsbut I want to make Cimi's job easy, too08:07
SaviqCimi, why do you say "repeater or listview are inefficient"? just for the sake of talking to the model I think that's fine08:07
SaviqCimi, the delegate would just be a QtObject08:08
CimiSaviq, so a listview with empty delegates?08:08
Cimiah ok08:08
Cimiand the flickable?08:08
mzanettiCimi: repeater, not listview08:08
SaviqCimi, why a flickable at all?08:08
SaviqCimi, and yeah, what mzanetti said08:08
CimiSaviq, listview has one08:08
Mirvtsdgeos: isn't it already?08:08
Mirvunity-scopes-api08:08
Saviqmzanetti, one thing ListView has over Repeater, though, is currentIndex/Item08:09
=== _salem is now known as salem_
mzanettiSaviq: but I guess in Cimi's case Repater.itemAt() is even more useful08:09
tsdgeosMirv: it has 0.4.5 when utopic has 0.4.708:09
Mirvtsdgeos: mm, weird08:09
Cimimzanetti, yup08:09
Saviqmzanetti, yeah, I mean rather that if an index goes away from a Repeater, you don't know08:09
tsdgeosMirv: what's even weirder is that i don't get the new version on dist-upgrade :S08:09
Saviqmzanetti, whereas in a ListView, it deals with it somehow08:10
mzanettiright...08:10
Saviqand you have currentItem, too08:10
Mirvtsdgeos: well of course because beta2 recipes have the epoch, to prevent that you'd suddenly get a non-rebuilt-against-qt5.3 version08:10
* tsdgeos pretends he understands that ^_^08:11
Mirvtsdgeos: I'm trying to open https://code.launchpad.net/~canonical-qt5-edgers/+recipe/daily-qt53-unity-scopes-api ... LP just sucks with recipe pages08:11
Mirvtsdgeos: hehe, the "1:" in front of the version08:11
CimiSaviq, currentItem relative to the user interaction on the flickable08:11
CimiI would not add a listview with all it comes from it just to get a model working08:12
Cimiinfographiocs with the current state should just be a crossfadeimage and one property with the url08:12
SaviqCimi, again, model is just future proof, and you're just throwing words "listview is inefficient" without even knowing what that means08:13
Saviqpete-woods, do you ever update the model or replace it whole every time?08:13
pete-woodsSaviq: only on first load08:14
pete-woodsafter that I only update the entries that actually change08:14
pete-woodsI tried to make it a proper list model08:14
CimiSaviq, it definitely adds a lot of stuff I don't need08:14
pete-woodsso your QML can be nice and efficient08:14
CimiSaviq, so we can call that a not efficient solution08:14
CimiSaviq, also, you really expect to swipe between infographics?08:15
CimiSaviq, we also swipe to unlock the lockscreen08:15
pete-woodsI think that'd be much nicer (vertical swipe)08:15
* mzanetti expect design to come up with something new in the infographic at least twice per year08:15
SaviqCimi, that was the design until we got "swipe the whole greeter away"08:15
Cimithat could collide UX wise08:15
SaviqCimi, that's the *only* reason why we're not swiping the infographics now08:16
SaviqI mean in what you're implementing now08:16
CimiSaviq, I have in mind the old design08:16
Saviqpete-woods, Cimi, ok, let's just add the good 'ol Q_INVOKABLE get(int index)08:16
CimiSaviq, when the infographics were like part of the dash?08:16
SaviqCimi, huh?08:17
pete-woodsSaviq: I've already got that method08:17
CimiSaviq, ok I am talking of the very first design...08:17
SaviqCimi, so you're talking tiles08:17
SaviqCimi, not infographics08:17
Saviqpete-woods, Q_INVOKABLE is it?08:17
pete-woodsyes08:17
CimiSaviq, they were infographics 3 years ago :)08:17
pete-woodsSaviq: this was at mzanetti's request :)08:18
CimiSaviq, issue is that I call get08:18
CimiSaviq, and the model is not ready08:18
CimiSaviq, like08:18
CimiSaviq, http://paste.ubuntu.com/7497997/08:18
CimiSaviq, source is empty08:19
SaviqCimi, yeah, which is why it's better to use something that understands updates on the model08:19
CimiSaviq, and I agree with that08:19
SaviqCimi, but I don't think moving it down to the plugin is the right thing to do08:20
CimiSaviq, well, I will trust you who know those things better than me08:20
CimiSaviq, let me try with repeater then08:20
Mirvtsdgeos: aaargh08:20
tsdgeosMirv: wha?08:21
Mirvtsdgeos: unity-scopes-api does not auto-build becaause of " Missing build dependencies: apparmor:native "08:21
Mirvtsdgeos: and that's a feature of the recipes part of LP. I hope the ":native" build dependencies don't spread too much08:21
SaviqCimi, yeah, remember you need to react to onCountChanged and check if your current index still "fits" in the model and such08:21
Mirvtsdgeos: so, I'll do a manual upload08:22
CimiSaviq, we don't have count on the plugin, though08:22
CimiSaviq, is count a qml property generated by repeater. right?08:23
SaviqCimi, http://qt-project.org/doc/qt-5/qml-qtquick-repeater.html#count-prop08:23
CimiI know that pro08:23
mhr3Mirv, Saviq wanted to look into it :)08:23
Cimijust wondering if it doesn't need that in the plugin too08:23
SaviqMirv, mhr3, we can revert the :native from -scopes-api, apparmor will be fixed to cope...08:24
mhr3Saviq, what about python3?08:24
Saviqmhr3, :any08:24
MirvSaviq: phew, nice. getting LP fixed would the best though.08:24
SaviqMirv, indeed08:24
Mirvit only complains about the :native stuff08:24
mhr3Saviq, also we're moving to libapparmor08:24
SaviqMirv, we did have that problem in jenkins, and fginther fixed it08:25
mhr3should workaround that08:25
Saviqmhr3, nice08:25
SaviqMirv, so maybe it wouldn't be that difficult to update LP08:25
Saviqpete-woods, Cimi, now, about remembering the last selected one... obviously without domain knowledge (what _is_ the one that's displayed now)08:25
Saviqpete-woods, Cimi, we can't reliably remember which one to display :|08:26
Ciminope08:26
MirvSaviq: maybe, there are just so few people working on LP nowadays08:26
Cimiwe need like an index08:26
Cimitoo08:26
Mirvtsdgeos: ok, new unity-scopes-api building, should be available in 30+ mins or so08:26
tsdgeostx08:26
Cimiso my starting index of the listview/repeater would be that index when we first launch unity08:27
SaviqCimi, "like an index"?08:27
CimiSaviq, the saved index08:27
SaviqCimi, yeah, but index isn't good enough08:27
mhr3Saviq, btw i saw some oddity with u8 yesterday, it stopped rendering and was using 100% cpu08:27
SaviqCimi, since the order might change08:27
mhr3Saviq, also, not reacting to any input08:28
CimiSaviq, indeed and this index in the model should point at the correct one, no?08:28
pete-woodsSaviq: indeed, but if you tell me to save an index, I can convert that to something persitent internally, though08:28
Saviqmhr3, not a great bug report08:28
Saviq;)08:28
Cimimhr3, you should remove your plastic cover from the display, might help on this 100% cpu usage08:29
mhr3Saviq, yea, sorry, things spinning in a loop are pain to debug08:29
Saviqpete-woods, yeah, but at that point the argument of moving more logic down to the plugin becomes more valid08:29
mhr3Cimi, :P08:29
mhr3Cimi, my work phone doesn't have it, you stripped it yourself08:29
CimiSaviq, otherwise I save the content of the last url of the svg, and loop in the model to find it?08:30
CimiSaviq, or maybe we can just add a search method to the model?08:30
SaviqCimi, I have no strong opinion there, but yeah something of the sort08:31
Saviqpete-woods, Cimi, I'll let you sort this one out08:31
fgintherSaviq, to be fair, the 'fix' was to strip ':native' since jenkins is already doing a native build08:32
Saviqfginther, oh ok...08:32
pete-woodsSaviq: I have no problem switching to the API I proposed earlier08:33
Saviqpete-woods, I wonder if we could combine them then08:33
pete-woodsSaviq: maybe, but that would probably start to make it convoluted08:34
pete-woodswhen we have a concrete use case right now08:34
pete-woodsI'd rather add complexity when it's actually necessary08:35
Saviqpete-woods, I know... I know... we can always go bzr revert at some point can't we...08:35
pete-woodsyep08:35
Saviqpete-woods, well, more work for you, then... and I'd actually still make the "new" api wrap the model anyway, but that's your call :)08:37
pete-woodsSaviq: if we need the old model, I'd just copy and paste the original implementation and have it as a separate class, most of the work is already done by a child class anyway08:37
Saviqpete-woods, mhm08:38
SaviqCimi, looks like the "remember last one shown" tipped the scale towards the simplified API08:38
Cimiyay08:39
pete-woodsCimi: I already have an implementation of the simple API for you, just fixing the unit tests08:39
tsdgeosguys, so i have this08:43
tsdgeoshttp://paste.ubuntu.com/7500659/08:43
tsdgeoswhere Department is a Qobject08:43
tsdgeosi was under assumption that JavaScript would delete the Department once i didn't reference to it in qml anymore08:43
tsdgeosbut it's not happening08:44
tsdgeoswhat i am doing wrong?08:44
Saviqtsdgeos, nothing, garbage collection needs to kick in08:44
tsdgeoshmmm, so you say it's not getting deleted because the GC didn't run?08:45
Saviqtsdgeos, I'd expect so08:45
tsdgeoscorrect08:45
Saviqtsdgeos, QML deals a bit different, though08:45
tsdgeosafter doing it a lot08:45
Saviqtsdgeos, if there's a delegate that gets destroyed, this should get deleted with it08:45
tsdgeosi got a bunch of "Department destroyed"08:45
tsdgeosso even the delegate was gone08:46
tsdgeosthe var that the delegate referenced stayed in the limbo for a while08:46
Saviqtsdgeos, yeah, that's unfortunately possible with GC, and I don't think we can do anything about it, other than call into C++ to destroy it on delegate destruction08:46
tsdgeosok, it's ok, it's not like we're going to have zillions of memory lost there08:47
Saviqtsdgeos, we could have a signal on Department that would be connected to in C++ and destroy it... stuff like that08:48
Saviqmhr3, but I believe Department objects will actually be rather light (proxies to a tree), mhr3?08:49
=== deegee__ is now known as drussell
Saviqtsdgeos, ↑ I meant08:49
tsdgeosyeah08:49
Saviqtsdgeos, for reference http://qt-project.org/doc/qt-5/qqmlengine.html#ObjectOwnership-enum08:51
tsdgeosSaviq: yes i was reading that08:51
tsdgeosbut then it wasn't doing what i expect it to do :D08:51
Saviqtsdgeos, well, according to that they will only get removed on engine deletion... :|08:51
tsdgeosi just expected the GC to kick in somewhat more often08:52
tsdgeoshmm, what?08:52
mhr3Saviq, it will have to dup the data, but still should be light-ish08:52
tsdgeosSaviq: that's the "An object tracked by one QQmlEngine" part08:52
Saviq"An object tracked by one QQmlEngine will be deleted during that QQmlEngine's destructor"08:52
tsdgeosyes, but if there's no references i assume it's not tracked by anyone08:53
tsdgeosno?08:53
Saviqhmm08:53
Saviq"QML will track it, and delete the object if there are no remaining JavaScript references to it and it has no QObject::parent()."08:53
tsdgeosi mean i'm getting the thing destroyed08:53
Saviqthat doesn't, unfortunately say *when* it's going to be deleted... but maybe it has to be like that08:53
Saviqtsdgeos, ah ok, just after some time?08:53
tsdgeosyep08:54
Saviqtsdgeos, then yeah, GC explains08:54
tsdgeoshttp://paste.ubuntu.com/7500687/08:54
tsdgeosthese qdebugs are all from the c++ side in constructor/destructor08:54
Saviqyeah sounds right08:55
* greyback heard of a "gc()" function that's supposed to force trigger the garbage collector08:55
tsdgeosmhr3: any ETA for the review of https://code.launchpad.net/~aacid/unity-scopes-shell/use-unity-api/+merge/219221 ?08:57
tsdgeosSaviq: or you ↑08:57
mhr3tsdgeos, opened it a minute ago :)08:57
tsdgeosah08:57
tsdgeosnice :)08:57
Saviqgreyback, indeed this seems mentioned here and there, can't find actual docs though08:58
greybackSaviq: yeah, nor have I, but did see it in Qt Components at one stage - in their PagStack they called it to immediately release resources held by a popped page08:59
mhr3tsdgeos, added comments09:07
Saviqgreyback, hah! http://qt-project.org/doc/qt-5/qtqml-javascript-qmlglobalobject.html09:08
Saviqgreyback, 3rd bullet point09:08
greybackSaviq: good google-fu :)09:08
Saviqterse09:08
tsdgeosmhr3: tx09:08
Saviqgreyback, grep over qtdeclarative rather ;)09:09
greyback:)09:09
Saviqtsdgeos, so yeah, if we decide we want to, we can call gc() after delegate destruction and the object should be dropped then and there09:11
tsdgeosok09:11
tsdgeosmhr3: answred them09:13
mhr3tsdgeos, hm, i see, can you add a comment to cmakelists about the moc issue?09:16
tsdgeosmhr3: sure09:16
tsdgeosmhr3: i guess we could run moc on the other side of the lib, but that's how we do it for the other unity-api stuff so i mimic'ed it09:19
tsdgeosmhr3: comment added09:19
mhr3tsdgeos, would require installing autogenerated moc... /me not likey09:20
tsdgeosSaviq: mhr3: silo!09:21
mhr3ah, right, we're not landing anything09:24
Cimipete-woods, tell me when you push09:24
pete-woodsCimi: will do09:25
Cimipete-woods, even without unit tests09:25
Cimipete-woods, I can work in the meanwhile :)09:25
pete-woodsCimi: sure, but I want to know it works09:28
pete-woodsI found it crashed on me once09:28
pete-woodsCimi: don't worry, I'll be quick09:28
* Cimi makes a coffee in the meanwhile :)09:28
pete-woodsmaybe not *that* quick!09:31
sil2100Saviq: hi! Do you know how's the split greeter going?09:37
sil2100Saviq: since there's a landing I wanted to assign that has indicator-network in it, can I ignore-conflicts or is this split ready for release soon?09:38
Mirvtsdgeos: unity-scopes-api up-to-date09:51
Mirvtsdgeos: unity-api is out of date though (fails tests on armhf, would need manual upload every time), and so is unity-scopes-shell (FTBFS since the last upload earlier this week, bug filed)09:53
tsdgeosoki09:55
Saviqsil2100, we're still a few issues away09:58
Saviqsil2100, so ignore it please and we'll manage09:58
mhr3Mirv, your fault :P https://bugs.launchpad.net/unity-scopes-shell/+bug/1321688/comments/110:01
ubot5Ubuntu bug 1321688 in unity-scopes-shell "unity-scopes-shell started FTBFS:ing after the last update, when used with Qt 5.3" [Undecided,Invalid]10:01
Mirvmhr3: oh so it's this same problem, that's actually great news! so the bug is invalid.10:02
Mirvso I'll just kick rebuild now 0.4.7 is there manually10:02
Mirvmhr3: no my fault, Launchpad's fault :)10:03
mhr3Mirv, sure, sure ;P10:03
Mirvso, unity-api only missing probably last update because it always fails on armhf + i386 (but not amd64...)10:03
Saviqthostr_, mhr3, I added a session on app installation progress in the app store/scope, who should be there? I added Rodney and you two from your side10:04
mhr3Saviq, +alecu & pstolowski10:05
Saviqdone10:05
pete-woodsCimi: okay, have pushed the branch now10:09
Cimithx¬10:09
Cimi!10:09
pete-woodsCimi: http://paste.ubuntu.com/7500892/ <— some example QML10:10
Saviqpete-woods, shall I rebuild the silo?10:10
pete-woodsSaviq: sure, can do that too10:10
pete-woodsI thought we'd got flushed10:10
Cimipete-woods, awesome!10:10
Saviqpete-woods, indeed we did10:11
Saviqpete-woods, we're low on silos still, so let's do it when we're ready10:11
pete-woodsthat works for me10:11
CimiI don't need it for testing it10:12
Cimibut phone would be good10:12
CimiSaviq, I guess crossbuild doesn't work yet?10:12
Cimitried yesterday and got issues10:12
SaviqCimi, you'd have to manually install build deps, it works then10:12
CimiSaviq, so I chroot and install deps?10:13
Cimiin sbuild10:13
SaviqCimi, yeah, instead of relying on sbuild to resolve the deps10:13
SaviqCimi, schroot -c foo-amd64-armhf -u root10:13
SaviqCimi, apt-get build-dep -aarmhf unity810:13
pete-woodsCimi: a quick warning, I think you'd going to find the CrossFadeImage looks a bit crappy10:13
pete-woodsat least it did for me10:13
Cimipete-woods, in which way?10:14
SaviqCimi, and resolve anything it complains about10:14
Cimipete-woods, I have a local version where I added transitions10:14
pete-woodsCimi: maybe you have improved it, but for me the transitions were pretty bad, like both images seemed to have 100% opacity at one point10:14
Cimipete-woods, fixed thay10:14
pete-woodsI think perhaps it's never been tested with transparent images?10:14
pete-woodsCimi: ah, cool, should be good then!10:15
CimiSaviq, are you in Malta?10:15
SaviqCimi, not until Saturday10:15
CimiSaviq, good for you10:15
CimiSaviq, I wanted to ask sdk guys how to name the property10:15
SaviqCimi, so bad? ;)10:15
CimiSaviq, for the crossfadeimage10:15
CimiSaviq, in the meanwhile I have10:16
Cimipastebinit qml/Components/CrossFadeInfographics.qml10:16
Cimihttp://paste.ubuntu.com/7500920/10:16
Saviq/food10:16
SaviqCimi, I'd go for "fadeStyle" or something - will allow supporting more modes later10:17
Cimidiff -urN ~/Development/sdk/ubuntu-ui-toolkit/modules/Ubuntu/Components/CrossFadeImage.qml qml/Components/CrossFadeInfographics.qml | pastebinit10:17
Cimihttp://paste.ubuntu.com/7500923/10:17
CimiSaviq, ok10:18
boikohi, any uoa devs around?10:22
boikomardy: hey, do you know of any changes released lately that could be breaking telepathy-ofono account creation on desktop and device?10:27
Mirvunity-scopes-shell built now too in qt5-beta2 PPA10:28
boikomardy: if I create the account with no parameters, it works fine, but as soon as I try to set modem-objpath it fails saying ofono/ofono doesn't have such property10:29
Mirvtsdgeos: I also now pushed the newest version of unity-api to build with tests disabled, in case it's needed. clearly at least the unity-scopes-shell was required to be together with new unity-scopes-api.10:32
tsdgeosty10:32
mhr3mzanetti, seen the unity-api test failure?10:36
mzanettimhr3: no. looking...10:36
mhr3mzanetti, https://launchpadlibrarian.net/175993802/buildlog_ubuntu-utopic-armhf.unity-api_1%3A7.81%2B14.10.20140520-0~134%2B201405201819~ubuntu14.10.1_FAILEDTOBUILD.txt.gz10:37
mzanettiuh... what the...10:37
* mzanetti fixing10:37
mhr3mzanetti, 5.310:37
mzanettiah10:37
mzanettimhr3: can we install that already without breaking everything?10:38
mhr3Mirv, ^10:38
=== salem_ is now known as _salem
Cimipete-woods, you mixed tabs and spaces in the modified files :)11:04
CimiInfographicsList.h11:04
pete-woodsCimi: thanks for pointing that out! I'm switching between two different projects styles :)11:04
Cimipete-woods, I personally prefer spaces in the end11:05
Cimibut here everything is tabs11:05
pete-woodsCimi: okay, I've fixed that, thanks for noticing11:05
pete-woodsI only really care about consistency11:06
CimiSaviq, so for our tests, I need to mock InfographicList with this new API, right?11:06
CimiI don't think I can do this in js11:06
mzanettimhr3: argh. can't reproduce there here with 5.311:07
pete-woodscan't you create a new QML type with a property and a slot?11:07
mhr3mzanetti, :(11:08
Cimipete-woods, mmm11:09
Cimipete-woods, I think is indeed possible11:10
Cimibasically wrapping11:10
Cimia listmodel11:10
Cimilet's see11:10
pete-woodsyeah, something like that11:10
Cimihaven't thought of that, thanks11:10
SaviqCimi, you don't even need to wrap a ListModel, an array would be good enough11:21
CimiSaviq, I did this11:21
CimiSaviq, http://paste.ubuntu.com/7501064/11:22
CimiSaviq, forget the next() function and the limited svgs I have now11:22
CimiSaviq, this way I have model.path and model.next()11:22
SaviqCimi, http://paste.ubuntu.com/7501070/11:24
SaviqCimi, it should be property *url* path actually11:24
Cimiok cppp11:24
Cimicool11:24
CimiSaviq, TypeError: Cannot read property 'path' of undefined11:28
Mirvmhr3: mzanetti yes, desktop (utopic) less tested though but QtC should continue to work11:28
Cimiforget me11:29
mzanettiMirv: thanks11:29
* Cimi hides11:29
SaviqCimi, I will11:29
SaviqCimi, really, IRC debugging is not useful :P11:29
Mirvinstructions at https://launchpad.net/~canonical-qt5-edgers/+archive/qt5-beta2 - I should write to the mailing list as well11:29
CimiSaviq, I'm trying to hide!!!!! :P11:29
Mirvnote the packages are "ugly" but nothing that should directly concern you. I'm working in parallel (and much slower) on "proper" packaging11:29
Saviqtsdgeos, looks like we got uitests hanging on jenkins :|11:30
Saviqtsdgeos, https://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-utopic/61/console11:30
Saviqtsdgeos, 3h is definitely too long11:30
tsdgeoswow11:30
tsdgeosaaaaaaaaaaactually it seems to hang here too :S11:31
tsdgeoswell it crashed :S11:31
tsdgeosSaviq: can you see if it crashes for you? i'm not sure if this crash is becasue 5.311:35
tsdgeoswhich is bad in itself11:35
Saviqtsdgeos, just all tests?11:35
tsdgeos./tests/plugins/Dash/OrganicGridTestExec11:35
tsdgeosSaviq: ↑11:35
Saviqtsdgeos, yup, fatal 1111:36
Saviqtsdgeos, in delegate creation ranges...11:36
tsdgeosthat's 5.2, no?11:38
tsdgeosSaviq: so what do we do? i go back to 5.2 and fix it? or given we don't use the organicgrid anywhere yet, just disable it? or?11:40
Saviqtsdgeos, yeah, I'm on 5.2... and we'll be enabling the other layouts asap, too11:40
Saviqtsdgeos, so...11:41
tsdgeosSaviq: ok, i guess the questions is we want to fix it in 5.2 or in 5.3?11:41
Saviqtsdgeos, 5.311:42
Saviqtsdgeos, http://paste.ubuntu.com/7501125/11:42
Saviqdied in GC ;|11:43
tsdgeosyep11:43
Saviqwonder if this is the same Olivier saw11:43
CimiSaviq, tsdgeos I have different rendering with qml and imageviewer/inkscape11:43
tsdgeosCimi: of course, i told you not to use SVG :D11:44
SaviqCimi, yeah, qt doesn't render svgs correctly11:44
tsdgeosthe svg render of Qt only renders in 1 pass11:44
tsdgeosso can't do lots of stuff11:44
SaviqCimi, we'll have to deal with that for now, and render them with librsvg in the image cacher11:44
CimiSaviq, but I receive svgs...11:46
SaviqCimi, yes, image provider will then render them into bitmaps and cache them11:47
SaviqCimi, until then, we'll have to live with not-so-great renders11:48
CimiSaviq, call them broken11:48
SaviqCimi, as we do in the dash for quite some time now11:48
CimiSaviq, missing elements11:48
Saviqbug #122499811:48
ubot5bug 1224998 in Unity 8 "Cache preview and thumbnail images" [Medium,Triaged] https://launchpad.net/bugs/122499811:48
CimiSaviq, if eog is using librsvg11:49
Cimiit is not good either11:49
CimiI have different results inkscape/eog11:49
tsdgeosSaviq: it may actually be an infinite loop somewhre in the new delegate range code, checking11:50
SaviqCimi, no idea what eog renders with11:50
SaviqCimi, but we need to decide on one that we say is the right one, and afaict librsvg was the one recommended implementation11:51
CimiSaviq, http://i.imgur.com/2ZMFNm8.png11:52
SaviqCimi, heh, that's bad ;)11:53
* tsdgeos lunches11:53
SaviqCimi, in any case, we know what the *target* renderer is11:53
CimiSaviq, inkscape uses cairo iirc11:53
SaviqCimi, so people that write the visualizers are expected to verify their SVGs work well in Qt/QML11:54
pete-woodsSaviq, Cimi: that's exactly what I did, yes11:54
Cimipete-woods, ok11:54
pete-woodsthe gnome render seemed pretty weak at rendering SVGs11:54
Cimipete-woods, this is an inkscape edited svg11:54
pete-woodsI just made sure it worked well in Qt11:54
pete-woodsoh, okay11:54
pete-woodsyou edited one of the ones that my program generated?11:55
Cimipete-woods, yes11:56
pete-woodscool11:56
pete-woodsglad the inkscape was happy with it11:56
pete-woods*that11:56
Cimipete-woods, not qt though :)11:56
pete-woodsah11:56
pete-woodsthe SVGs I generate are pretty clean11:57
Cimipete-woods, I don't know how people are supposed to create those infographics svgs11:57
pete-woodsyou could always tweak them by hand11:57
pete-woodsCimi: well realistically it's "write a script"11:57
pete-woodsbut yes, I understand completely what you're saying11:57
Cimiwhat's the point of using svgs if to edit them I need to write code?11:58
pete-woodsCimi: agreed11:58
Cimiwe could have just used qml or css11:58
pete-woodsyep11:58
SaviqCimi, no we couldn't11:58
CimiSaviq, ok for security11:58
Saviqexactly11:58
pete-woodsit's still a completely rubbish development experience for someone who wants to write an infographic11:59
CimiI agree11:59
Saviqpete-woods, Cimi, you're free to come up with something better11:59
mzanettimhr3: https://code.launchpad.net/~mzanetti/unity-api/fix-launcher-tests-with-5.3/+merge/22062311:59
pete-woodsSaviq: I think even a library to help write SVGs would help12:00
pete-woodsI ended up just using QXmlStreamWriter12:00
Saviqpete-woods, you can use whatever library you want12:00
CimiSaviq, pete-woods I don't know what this script it, but it feels to me it would be easier to do other things :)12:00
=== _salem is now known as salem_
Cimiit's like a new language to learn12:00
Saviqpete-woods, provided you ship it with your visualizer and there's just an executable that spits the svg out12:00
CimiI don't expect any designer learning to code to draw an infographics12:00
SaviqCimi, designers are not meant to code them, they are meant to design them12:01
pete-woodsCimi: http://bazaar.launchpad.net/~unity-team/libusermetrics/file-based-infographics/view/head:/src/infographic/Infographic.cpp12:01
pete-woodsthat's the code that makes the default infographic in-case you're interested12:01
pete-woodsit's not like it's intractable, you just need to understand SVG12:02
Saviqpete-woods, in time we'll bless a library that will become part of our SDK12:02
Saviqpete-woods, until then, use whatever floats your boat12:02
pete-woodsthat'd go a long way towards helping12:02
pete-woodsyeah, I know *I* can do whatever I want12:02
pete-woodsit's just making it nice for other people I worry about12:02
pete-woodsanyway, I implemented the things as I was told, and here it is! :)12:03
pete-woodsjust wish there was some way to embed a sandboxed lump of QML into the greeter12:03
Saviqpete-woods, I think giving people the freedom is better than to say "here's the library to use", people are quite happy with choosing to do what they want12:03
pete-woodsI don't mean to say they have to be restricted12:04
Saviqaaanyway12:04
pete-woodsjust some help along the way, like a library with classes representing the different SVG types12:04
pete-woodsyep12:04
pete-woodsit's all done now12:04
pete-woodsso no point discussing12:04
=== MacSlow is now known as MacSlow|lunch
CimiSaviq, the original idea of using SVGs was "so designers can draw what they want" - which is not what will happen12:05
CimiSaviq, I understand though the security benefits now12:06
SaviqCimi, that idea was shot down years ago12:06
SaviqCimi, because it's just impossible12:06
=== om26er_ is now known as om26er
MirvSaviq: that qtubuntu patch fixes half of the world12:27
Mirvlike all ~rendering issues while the crash issues remain12:27
SaviqMirv, yup, that's expected :)12:27
SaviqMirv, is there a bug for settings app being empty?12:35
MirvSaviq: no there isn't, it hasn't been debugged so far as to get eg. some error output to quote12:37
SaviqMirv, I'll do12:37
MirvSaviq: great, thanks!12:37
Mirvit's possibly related to some of the UITK issues seen in the unit tests12:38
Mirvalso, performance of Qt 5.3 in Unity 8 side-by-side is visibly better12:38
MirvI didn't realize it before davmor2 showed his 5.2 mako and we eg. scrolled app scope at the same time12:39
SaviqMirv, orly? nice12:41
SaviqMirv, bug #132215912:41
ubot5bug 1322159 in ubuntu-ui-toolkit (Ubuntu) "Pages empty under Qt 5.3" [Undecided,New] https://launchpad.net/bugs/132215912:41
Mirvthx12:41
SaviqMirv, maybe his 5.2 was before the scope performance improvements? ;)12:41
Mirvdavmor2: were you running ~latest on your mako?12:42
davmor2Mirv: yes12:43
davmor2Mirv: r4312:43
=== zequence_ is now known as zequence
MirvSaviq: ^ that's what I thought too, but it was worth checking12:50
Mirvthanks dave12:50
=== MacSlow|lunch is now known as MacSlow
tsdgeosSaviq: soooo, it's actually two things13:34
tsdgeosthere's a bug in porting the test13:34
tsdgeosand then there's a bug in the code that makes the "new" test crash13:34
Cimitsdgeos, Saviq can I define enums from qml?13:35
SaviqCimi, no13:35
SaviqCimi, go for strings13:35
CimiSaviq, ok13:35
tsdgeosSaviq: so i'll fix both :D13:38
tsdgeosSaviq: or c++ enums, no?13:38
Saviqtsdgeos, cool you found it :)13:38
Saviqtsdgeos, it's a QML-only API (CrossFadeImage)13:39
tsdgeosi see13:39
Saviqtsdgeos, we *could* expose the enums in some global object from C++ in Ubuntu.Components, but I don't think there is something like that yet13:39
Saviqso let's see what the SDK team says13:40
Cimi#namingishard13:41
CimiFadeOutFadeIn?13:41
CimiSaviq, we have two images13:42
SaviqCimi, parallel, sequential13:43
Cimiright13:43
CimiSaviq, for the current one?13:43
Cimione fades in13:43
SaviqCimi, overlay?13:44
Cimigood one13:44
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/organiccgridcrash/+merge/22065113:51
Saviqtsdgeos, we have a leftover CardHeader entry in tests/qmltests/CMakeLists.txt :/14:13
tsdgeoswops14:13
tsdgeosSaviq: i'll do a quick MR to kill it14:14
Saviqtsdgeos, tx14:14
tsdgeosSaviq: https://code.launchpad.net/~aacid/unity8/nomorecardheader/+merge/22065914:18
Saviqyikes that's bad... over an hour and it's still building??14:20
Saviqhttp://s-jenkins.ubuntu-ci:8080/job/unity-phablet-qmluitests-utopic/62/consoleFull14:20
Saviqjeez we need timestamps in jenkins :(14:21
CimiSaviq, ok infographics working fine14:23
CimiSaviq, only issue I have is testing on the desktop14:23
CimiSaviq, where UID is 100014:23
Cimiand we're using mocks for lightdm14:24
SaviqCimi, testing as in actual real testing?14:27
CimiSaviq, using on the desktop14:27
SaviqCimi, just tweak the mocks14:27
=== tedg is now known as ted
SaviqCimi, so that it comes up with the right UID14:27
CimiSaviq, mocks for what?14:27
SaviqCimi, LightDM14:28
Cimiyes but the mock isn't for the phone as well?14:28
Cimiand on the phone has different uid14:28
Cimihttps://code.launchpad.net/~cimi/unity8/infographics-new-lightdm/+merge/220662 in the meanwhile14:28
CimiI have to test it on tablet too14:29
SaviqCimi, I didn't mean to tweak it and commit it14:29
Saviqjust tweak it for testing14:30
mhr3mzanetti, ah... parentless qobjects... nasty14:35
mhr3guess they changed how gc runs14:35
mzanettimhr3: the nasty thing is that Qt < 5.2 didn't take ownership of those, while >= 5.2 does14:35
mzanettimhr3: not sure why we only see the crash with 5.3 now14:35
mzanettiyeah... probably14:35
mhr3mzanetti, according to docs qml takes ownership of such objects since 5.014:36
mhr3mzanetti, at least i remember reading about it in 5.0 docs14:36
mzanettimhr3: it probably was just buggy in 5.014:36
mzanettimhr3: xbmcremote started to go all wonky wiht 5.2 only14:36
mhr3or as i said.. delayed :)14:36
CimiSaviq, but when I run unity8, it uses lightdm mock afaics14:37
dednickcharles: ping14:38
mzanettimhr3: no... with 5.0 it wasn't delayed even... I would have noticed that14:38
mzanettiunless it was delayed for days :D14:38
mhr3mzanetti, perhaps delayed until QQmlEngine was about to be destroyed ;)14:39
mzanettiSaviq: https://bugs.launchpad.net/unity8/+bug/130916214:39
ubot5Ubuntu bug 1309162 in unity8 (Ubuntu) "Unity8 crashed with sigsegv after touching on an alarm from the upcoming indicator (sync in progress)" [Undecided,New]14:39
mzanettimhr3: yeah... that could have been, yes14:39
SaviqCimi, yes, because it can't use the actual lightdm plugin, 'cause that's only allowed for the greeter14:41
SaviqCimi, not sure what you want to achieve?14:42
CimiSaviq, I run unity on the desktop, lightdm mock gives me uid 100014:42
Saviqmzanetti, tx14:43
CimiSaviq, I run unity on the phone, 3201114:43
SaviqCimi, you might want to tweak the mock to get the actual UID, although I wonder if we'll need a mock of the infographics backend anyway, don't think it should be available in the user session anyway (just for greeter)14:44
SaviqCimi, but I wouldn't care much about this now14:45
SaviqCimi, as long as you have it tested in isolation14:45
CimiSaviq, so as it is now, should work on the phone14:45
SaviqCimi, and verified it works for real on the phone14:45
SaviqCimi, that's good enough14:45
CimiI have to test it again14:46
SaviqCimi, the "windowed" unity8 isn't our top prio14:46
Cimiworks on desktop if I manually set uid in GreeterContent.qml;14:46
SaviqCimi, and when you run it with -f, it will load the mock infographics one anyway won't it?14:46
SaviqCimi, good enough14:46
CimiSaviq, we removed the mock infographics14:46
CimiI merged your branch14:47
SaviqCimi, didn't you just write a new mock infographic backend?14:47
CimiSaviq, I don't14:47
SaviqCimi, after mterry lands his greeter, you won't even see the greeter when running unity8 anyway, so14:47
CimiSaviq, for tests I wroto those two qml lines14:47
SaviqCimi, what's that then http://paste.ubuntu.com/7501070/ ?14:47
CimiSaviq, it's the mock in the tests14:48
SaviqCimi, yeah, how are you passing it to GreeterContent?14:48
Cimiactually I haven't tested greeterContent14:48
CimiSaviq, I am passing to Infographics14:48
SaviqCimi, we need this to be an object under the same import as the original one14:49
SaviqCimi, otherwise wherever you go "import Infographics", it will load the actual backend - we don't want that14:49
Cimiok14:49
Saviqmhr3, do you have the :native → :any change MP'd somewhere? or shall I?14:59
mhr3Saviq, pls do, you're the ma expert :)14:59
Saviqlol14:59
mhr3Saviq, but you pete's branch as base15:00
mhr3s/you/use/15:00
mhr3Saviq, lp:~unity-api-team/unity-scopes-api/use-libapparmor15:00
Saviqmhr3, kk15:02
SaviqCimi, once you're done with infographics, could you please look at https://bugs.launchpad.net/unity8/+bug/132223315:10
ubot5Ubuntu bug 1322233 in Unity 8 "Greeter tests fail and can't be discerned" [Critical,Triaged]15:10
=== boiko_ is now known as boiko
dandraderCimi, can you approve it now? https://code.launchpad.net/~dandrader/unity8/unusedBackgroundImage/+merge/22028815:32
=== gatox is now known as gatox_lunch
=== greyback is now known as greyback|afk
=== dandrader is now known as dandrader|afk
=== kenvandine_ is now known as kenvandine
=== dandrader|afk is now known as dandrader
=== greyback|afk is now known as greyback
popeywillcooke: hello!16:17
willcookeoh hai!16:18
willcookeI'm going to try and write a scope.  Be afraid.16:19
willcookeDo I have to learn C++ first?16:19
* willcooke thinks this might take longer than a weekend16:19
popeyyes16:21
davidcallewillcooke, Go works too16:21
popeyand maybe go in the future16:21
popey\o/ We're in the future!16:21
davidcallewillcooke, or (don't tell anyone) Python.16:21
willcookehrm.16:21
willcookewait wat!16:21
willcookePython eh16:21
willcookeOk, now we're cooking16:21
willcookewell, we're warming things up in the microwave, but it's kinda the same thing16:22
davidcallewillcooke, look at lp:ubuntu-rest-scopes's README, all you need to know16:22
willcookethx davidcalle!16:22
davidcallewillcooke, my pleasure. BUT, these won't ever be client side scopes, only smarscopes server side.16:23
* willcooke reads about smart scopes16:23
davidcallewillcooke, device side is, only Go and C++.16:23
popeyi think willcooke is looking at desktop (unity 7) not device (unity 8)16:24
willcookeI'm easy16:24
willcookeI wonder if I can learn Go enough to copy&paste some examples16:24
willcookeprobably16:24
dandradergreyback, with qt compositor stuff on top of latest image I'm not getting any icons in the dash, only their titles and sections16:24
dandradergreyback, are you getting the same?16:24
greybackdandrader: same here, yes16:25
davidcallepopey, willcooke, aaaah... Then it's C++. Go scopes won't work for Unity7.16:25
dandraderwonder if rebasing on top of latest lp:unity8 will be of any help16:26
davidcallewillcooke, in any case, there is a soundcloud go scope (for unity8) in lp unity-scope-soundcloud, if you want to have a look16:26
willcookeI'll take a look.  What I'd like to implement is pretty simple.  I'll do some reading and see if I can make it work.  Thanks davidcalle16:26
davidcallewillcooke, np16:27
=== gatox_lunch is now known as gatox
dandradergreyback, hah! rebasing on top of latest trunk solved it. pushed.16:36
greybackdandrader: yay16:36
greybackdandrader: had a breakthrough with the resizing problem too16:37
dandradergreyback, awesome. so, what's happening?16:37
greybackdandrader: hmm, maybe postpone that. Breakthrough not complete16:41
=== salem_ is now known as _salem
mhr3Saviq, is there in integration u8 branch for the unity-api+scopes-plugin changes?17:47
mhr3seems it's far from working17:47
Saviqmhr3, https://code.launchpad.net/~aacid/unity8/use-unity-api/+merge/21922217:47
mhr3someone forgot to add that to the landing17:48
* mhr3 < someone17:48
Saviq;-)17:48
=== dandrader is now known as dandrader|lunch
=== alan_g is now known as alan_g|EOD
=== gatox_ultra is now known as gatox
=== dandrader|lunch is now known as dandrader
=== dandrader is now known as dandrader|afk
=== dandrader|afk is now known as dandrader
=== jhodapp is now known as jhodapp|afk
=== _salem is now known as salem_
=== salem_ is now known as _salem

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