Minutes Monday Plasma Hangout

[Bhushan Shah]

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
-------------- next part --------------
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170808/68eb99fa/attachment.sig>