Sorry for delayed note-sending, attached logs and notes -- Bhushan Shah http://blog.bshah.in IRC Nick : bshah on Freenode GPG key fingerprint : 0AAC 775B B643 7A8D 9AF7 A3AC FE07 8411 7FBC E11D
2017-08-07 15:31:47 <@bshah> meeting? 2017-08-07 15:32:02 <notmart> meetng! 2017-08-07 15:32:27 <@bshah> notmart: d_ed: kbroulik: meeting time! 2017-08-07 15:32:32 <kbroulik> here 2017-08-07 15:33:17 <@bshah> alright.. notmart starts then 2017-08-07 15:33:25 <notmart> ok 2017-08-07 15:33:35 <notmart> * kirigami toolbar component: fixing issues when transitioning between single and multiple column mode, experimenting ways to put extra custom widgets in that toolbar 2017-08-07 15:33:36 <notmart> * kirigami, high dpi fixes, correct sizing for handles and action buttons, better rendering of labels on non integer scaling 2017-08-07 15:33:38 <notmart> * desktop qqc theme: better rendering of text on non integer scaling 2017-08-07 15:33:39 <notmart> * wayland: start of implementation of the xdg_foreign protocol 2017-08-07 15:33:41 <notmart> * wayland: investigating a crash in ping in the xdgv6 protocol, leading to another bug uncovering 2017-08-07 15:33:42 <notmart> * fixed undeterministic ordering in modules order 2017-08-07 15:33:44 <notmart> * patch for bug in notifications with multi screen https://phabricator.kde.org/D7127 2017-08-07 15:33:44 <sKreamer> Diff 7127 "ignore spurious resize events to empty sizes" [Needs Revision] https://phabricator.kde.org/D7127 2017-08-07 15:34:03 <kbroulik> "desktop qqc theme: better rendering of text on non integer scaling" ← how's that? 2017-08-07 15:35:41 <notmart> sadly, atm is just by using distance filed rendering on non integer scalings 2017-08-07 15:35:47 <kbroulik> right 2017-08-07 15:36:04 <notmart> which, doesn't look gorgeous, but at least not completely garbled 2017-08-07 15:36:28 <d_ed> there's a few QtQuick scaling changes recently 2017-08-07 15:36:32 <d_ed> which might help 2017-08-07 15:36:43 <notmart> in 5.9 branch? 2017-08-07 15:37:09 <notmart> (which makes me think, still no answer of the parenting stuff take 2 in qt, meh) 2017-08-07 15:38:18 <@bshah> I guess it's turn of kbroulik then 2017-08-07 15:38:23 <kbroulik> * Did some model cleanup in FolderView (Pro tip: implementing setData in QAIM makes "model" magic property writable from QML) 2017-08-07 15:38:36 <kbroulik> * Various KIOSK-related fixes for GHNS, "download new wallpaper" and "download new plasmoids" ghns button is now hidden if "ghns" restriction applies 2017-08-07 15:38:44 <kbroulik> * Fixed a visual glitch when expanding/collapsing treeviews in Breeze widget style 2017-08-07 15:38:54 <kbroulik> * Fixed URLs in notifications not clickable. Selecting text inside them is still broken but I don't think we can fix that (we have couple conflicting interactions there: clicking links, selecting text, triggering the default action, flicking, ..) 2017-08-07 15:39:04 <kbroulik> * I might look into hybrid suspend support, if I can figure out how to make Logind allow "CanHybridSleep". I think the intermediate solution would be to just add a new entry in kickoff and powerdevil, since it's not something that is enabled by default in most distros, no? A proper solution would perhaps be to add an option to PowerDevil to have suspend-to-ram or suspend-to-disk instead do HybridSuspend? 2017-08-07 15:39:05 <notmart> kbroulik: you mean in the delefate model.roleName = "blah" will call setData? 2017-08-07 15:39:48 <kbroulik> yes 2017-08-07 15:39:52 <notmart> neat 2017-08-07 15:40:14 <kbroulik> pretty cool, works also across qsortfilterproxymodels etc 2017-08-07 15:40:27 <kbroulik> makes for really neat code, and saves custom Q_INVOKABLE setFoo(int index, ...) 2017-08-07 15:40:46 <notmart> sometimes even a pleasant surprise, go figure :) 2017-08-07 15:41:40 <@bshah> if kbroulik is done it's turn of d_ed 2017-08-07 15:42:09 <d_ed> I've been away most of last week, I've written up notes of my time at Guadec, which I think are worth reading 2017-08-07 15:42:22 <d_ed> Also we /possibly/ have a lead on that stupid AnimationFinished crash 2017-08-07 15:42:43 <kbroulik> that uncontrolledAnimationFinished? 2017-08-07 15:42:43 <notmart> wa reading and writing some questions as followup 2017-08-07 15:42:50 <d_ed> kbroulik: yes that 2017-08-07 15:42:52 <notmart> ooh, nice, what is it? 2017-08-07 15:42:58 <d_ed> I'm trying to make a smaller test. 2017-08-07 15:43:32 <d_ed> we have a valgrind trace of a double free, when we have nested grouped animations; it seems to be us disabling one of the animations mid-animation. 2017-08-07 15:43:46 <d_ed> so when the group finishes it deletes an already deleted thing 2017-08-07 15:44:31 <fvogt> Did someone bisect already as it's IIRC a regression? 2017-08-07 15:44:47 <Sho_> oh damn 2017-08-07 15:44:49 <Sho_> i forgot the meeting 2017-08-07 15:44:54 <Sho_> i'm here too 2017-08-07 15:44:55 <d_ed> I will when I have a reproducible case 2017-08-07 15:45:12 * Sho_ even has an exciting item 2017-08-07 15:45:16 <d_ed> talk of Qt regressions, I am also somewhere on the slideshow leaking huge amounts of memory 2017-08-07 15:45:49 <d_ed> we're meant to go "render window 1, render window 2, all windows are done I'll release resources" 2017-08-07 15:46:07 <d_ed> somehow we go "render window 1, render window 2, oh look! window 1 needs rendering, I'll do that" and so on 2017-08-07 15:46:15 <d_ed> I think that's also our high CPU usage bug too, all in one 2017-08-07 15:46:28 <notmart> because the panel overlaps the desktop? 2017-08-07 15:46:31 <fvogt> Maybe, we reverted the offending commit in our packages and the mem leak is gone 2017-08-07 15:46:37 <fvogt> The CPU usage remains the exact same 2017-08-07 15:47:08 <Sho_> d_ed: reading your wayland notes, awesome stuff, i had no idea you were going there! 2017-08-07 15:47:22 <d_ed> notmart: I doubt it as rendering windows is fairly independent of the final position 2017-08-07 15:47:22 <kbroulik> Sho_: so, d_ed's ninja stealth mode indeed worked :P 2017-08-07 15:47:42 <Sho_> :) 2017-08-07 15:47:47 <notmart> yeah, would look weird 2017-08-07 15:47:59 <d_ed> fvogt: do you have it reproducible? 2017-08-07 15:48:40 <d_ed> anyway, that's my main focus - those two bugs 2017-08-07 15:48:51 <d_ed> and I'm done talking 2017-08-07 15:48:53 <@bshah> okay, Sho_'s turn 2017-08-07 15:49:21 <notmart> d_ed: also: you solved the ping crash in xdgv6 branch.. would make this mergeable now' 2017-08-07 15:49:25 <Sho_> * [TM] Finally fixed the infamous, often-reported QSortFilterProxyModelPrivate::updateChildrenMapping crasher, the only significantly-reported libtm bug! Phew 2017-08-07 15:49:26 <Sho_> * [TM] Window tasks that hit the fallback codepath of using the window icon now update the icon as the window changes it, avoiding "icon updates only randomly" user experience 2017-08-07 15:49:28 <Sho_> * [TM] Finished & merged patch to fix regression breaking resizing and closing the group popup dialog as windows are closed and a group dissolved, plus more graceful resizing only after animations end 2017-08-07 15:49:30 <Sho_> * [Launchers] Tested the latest revision of the KAStats-based favorites patch, and it seems good for merge now 2017-08-07 15:49:31 <Sho_> * [General] I think I can get back to Wayland now + dabble a little more in KUserFeedback on the side 2017-08-07 15:49:32 <Sho_> * [Other] Spent quite some trying to mediate a team dispute this week 2017-08-07 15:49:34 <Sho_> * [Other] I'll be in Berlin in mid-October to shepard KDE's presence at QtWS'17 and to attend a KDE e.V. board meeting 2017-08-07 15:50:44 <d_ed> notmart: yeah, I updated the diff this morning 2017-08-07 15:51:03 <Sho_> next() 2017-08-07 15:51:09 <notmart> yep, i did read the diff 2017-08-07 15:51:12 <notmart> let's see what martin says 2017-08-07 15:51:13 <@bshah> okay,my turn 2017-08-07 15:51:31 <@bshah> - Helping postmarketOS people getting kwin running, so eventually they will have plasma mobile :> 2017-08-07 15:51:31 <@bshah> - Still fighting with pulseaudio droid and droidglue packaging 2017-08-07 15:51:31 <@bshah> - Got my one plus 5 delivered so I'll be able to test and work on images for devices with QTI_BSP 2017-08-07 15:51:49 <@bshah> questions? or next 2017-08-07 15:51:55 <d_ed> what is QTI_SEKRHAER 2017-08-07 15:52:23 <@bshah> d_ed: it's define in hwcomposer header which changes the ABI of hwcomposer 2017-08-07 15:52:34 <d_ed> ah 2017-08-07 15:52:36 <@bshah> it is used by non-nexus qualcomm devices 2017-08-07 15:54:10 <kbroulik> sounds like Paamayim Nekudotayim 2017-08-07 15:54:34 <@bshah> huh? 2017-08-07 15:54:40 <d_ed> a hebrew PHP error 2017-08-07 15:54:48 <d_ed> which is a very obscure reference 2017-08-07 15:55:15 <@bshah> hm 2017-08-07 15:55:32 <@bshah> anyway.. meeting is done.. I'll send notes soon.. \o/ 2017-08-07 15:55:35 <@bshah> Happy hacking!
notmart: * kirigami toolbar component: fixing issues when transitioning between single and multiple column mode, experimenting ways to put extra custom widgets in that toolbar * kirigami, high dpi fixes, correct sizing for handles and action buttons, better rendering of labels on non integer scaling * desktop qqc theme: better rendering of text on non integer scaling * wayland: start of implementation of the xdg_foreign protocol * wayland: investigating a crash in ping in the xdgv6 protocol, leading to another bug uncovering * fixed undeterministic ordering in modules order * patch for bug in notifications with multi screen https://phabricator.kde.org/D7127 kbroulik: * Did some model cleanup in FolderView (Pro tip: implementing setData in QAIM makes "model" magic property writable from QML) * Various KIOSK-related fixes for GHNS, "download new wallpaper" and "download new plasmoids" ghns button is now hidden if "ghns" restriction applies * Fixed a visual glitch when expanding/collapsing treeviews in Breeze widget style * Fixed URLs in notifications not clickable. Selecting text inside them is still broken but I don't think we can fix that (we have couple conflicting interactions there: clicking links, selecting text, triggering the default action, flicking, ..) * I might look into hybrid suspend support, if I can figure out how to make Logind allow "CanHybridSleep". I think the intermediate solution would be to just add a new entry in kickoff and powerdevil, since it's not something that is enabled by default in most distros, no? A proper solution would perhaps be to add an option to PowerDevil to have suspend-to-ram or suspend-to-disk instead do HybridSuspend? d_ed: * I've been away most of last week, I've written up notes of my time at Guadec, which I think are worth reading * Also we /possibly/ have a lead on that stupid AnimationFinished crash * I'm trying to make a smaller test. * we have a valgrind trace of a double free, when we have nested grouped animations; it seems to be us disabling one of the animations mid-animation. * so when the group finishes it deletes an already deleted thing Sho_: * [TM] Finally fixed the infamous, often-reported QSortFilterProxyModelPrivate::updateChildrenMapping crasher, the only significantly-reported libtm bug! Phew * [TM] Window tasks that hit the fallback codepath of using the window icon now update the icon as the window changes it, avoiding "icon updates only randomly" user experience * [TM] Finished & merged patch to fix regression breaking resizing and closing the group popup dialog as windows are closed and a group dissolved, plus more graceful resizing only after animations end * [Launchers] Tested the latest revision of the KAStats-based favorites patch, and it seems good for merge now * [General] I think I can get back to Wayland now + dabble a little more in KUserFeedback on the side * [Other] Spent quite some trying to mediate a team dispute this week * [Other] I'll be in Berlin in mid-October to shepard KDE's presence at QtWS'17 and to attend a KDE e.V. board meeting bshah: - Helping postmarketOS people getting kwin running, so eventually they will have plasma mobile :> - Still fighting with pulseaudio droid and droidglue packaging - Got my one plus 5 delivered so I'll be able to test and work on images for devices with QTI_BSP
signature.asc
Description: PGP signature