[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Minutes Monday Plasma Hangout
From:       Eike Hein <hein () kde ! org>
Date:       2017-12-11 13:29:11
Message-ID: 0d28ccd8-0b17-d7d6-21e9-6c64994739a3 () kde ! org
[Download RAW message or body]

Minutes! Logs! We got it all!

Only available as bundle.


Cheers,
Eike

["minutes-week-50.txt" (text/plain)]

Plasma Team meeting, 11-12-2017

Present: romangg, d_ed, notmart, bshah, Sho_

romangg:
Pushed Night Color / LibColorCorrect patches to KWin, plasma-workspace and \
plasma-desktop. Testers welcome :) There is currently still a discussion about if the \
kcm should be shown on X or hidden. It currently looks like this on X: \
https://imgur.com/a/sYBdQ I'm for showing it because (a) users know this way that \
this functionality exists in general (b) we won't get hit with dozens of bug reports \
asking where they can activate this functionality they heared about on omgubuntu / \
phoronix. (c) gives user small push to try out wayland

d_ed:
- Working on some startup improvements
- With aleix, we fixed a deadlock in ksmserver which has a huge impact on everyone
- Then lots of minor things related to that
- I need some reviews on me removing multihead

notmart:
* almost kill the packageurlinterceptor usen in plasma, seems speed up things a bit: \
                https://phabricator.kde.org/D9124
* kpackage loading binary rcc files: performance increase looks promising, patches \
                all over the place, most pushed, still to go:
** https://phabricator.kde.org/D9197
** https://phabricator.kde.org/D9237
* global menu: make it depening on available visualizations (plasmoid or kwin button) \
                not internal settings:
** https://phabricator.kde.org/D9215
** https://phabricator.kde.org/D9216
** https://phabricator.kde.org/D9267
* starting to look into adding autostart of apps support in look and feel, for \
latte-dock (or plank, or conky, or whatever sillyness users want)

bshah:
- Fixed plasma mobile images
- Some bugs which I need help with
    - huge scrollbars
    - broken struts
- working on packaging multimedia bits will have next image with it

Sho_:
* Wrote a patch for Plasma::Svg not to tear down the renderer and repaint when \
setImagePath is called with the same arg again, speeds up both plasmashell startup \
                (0.5%) and Simple Menu opening
* Wrote another patch for Plasma::Svg to cache the scale factor to initialize with in \
a static member to work being done twice on systems with non-1x scaling, speeds up \
                plasmashell startup by 2%
* Spent some more time banging on Plasma::Svg, chased down some rabbit holes, didn't \
                find anything too promising
* Implemented QQmlParserStatus in FolderModel, saves some churn during initialization
* Wrote a patch to reduce vertical gutter width in Folder View icon views based on \
                user feedback, needs more work though
* Wrote an optimization patch for the panel containment, saving one object per applet
* Wrote a patch for Plasma::FrameSvgItem avoiding an update during instanciation \
                before the component is complete
* Wrote a patch for KNotification to lazy-instanciate & lazy-load plugins, speeds up \
login time as well as mem usage of basically all apps, as well as shortens latency of \
their first notification event - helps with Plasma startup speed via kded \
                notifications
* Moved KGamma init to the right autostart phase to speed up login a little bit
* Came up with a plan for how to move KAccess out of early startup, make more of it \
work on Wayland and kill its permanent process - the bell goes into a KWin effect, \
the screen reader toggle action goes into a kded module, the rest goes into regular \
                autostart
* Minor improvement to existing KAccess KCMInit to save a config file read during \
                login
* Wrote patch to speed up plasmashell startup and save a little memory by not setting \
                a window icon (it's not needed due to NET::WM2DesktopFileName)
* Resurrected David's patch speeding up Plasma::Dialog and saving some memory by not \
                setting a window icon (it's not needed due to \
                NET::WM2DesktopFileName)
* Wrote patches to speed up xembedsniproxy, kactivitymanagerd and kglobalaccel5 \
                startups by not loading KDE platform module
* Wrote a patch for the Virtual Desktops KCM to make the number-of-rows field to have \
                the correct lower bound of 1 instead of 0 (user bug report came in \
                during the day)
* Ported launch feedback KCM rewrite to new base items from Marco's patch, done from \
                my end
* Sped up Simple Menu opening by reducing delegate complexity and reworking delegate \
                pre-caching
* Reworked Simple Menu keyboard navigation:
** Arrow nav after enterign search terms works again
** Horizontal arrow nav through search results works immediately, no need to arrow \
                down into the grid first
** Tab/Shift+Tab now work to cycle through the search field, the system favorites and \
                the category list
* Released Simple Menu v1.0.5 and v1.0.6 (performance work + keyboard work + hotfix)
* Worked around a Qt bug causing a scroll indicator line to appear at the bottom of \
Simple Menu pages (which can't be scrolled vertically ...), fix is in \
                plasma-framework
* Ported ksystraycmd to KF5 (https://cgit.kde.org/scratch/hein/ksystraycmd.git/)
* Non-Plasma: Improved Dolphin startup speed on systems without kio-stash installed \
                (i.e. most) by using a faster way to determine whether it is \
                installed or not
* Personal: On vacation Dec 18th-Jan 2nd


Log:

‎[20:04] ‎<‎Sho_‎>‎ MEETING TIME
‎[20:04] ‎<‎Sho_‎>‎ Who's here
‎[20:04] ‎<‎bshah‎>‎ Sho_: sorry, best you do it, internet bit flaky
‎[20:04] ‎<‎d_ed‎>‎ I am here
‎[20:04] ‎<‎Sho_‎>‎ sure
‎[20:04] ‎<‎notmart‎>‎ here
‎[20:04] ‎<‎Sho_‎>‎ note: everyone who's not here is by implication not a \
cool kid ‎[20:04] ‎<‎romangg‎>‎ here
‎[20:04] ‎<‎romangg‎>‎ puh
‎[20:04] ‎<‎Sho_‎>‎ :)
‎[20:04] ‎<‎Sho_‎>‎ kbroulik? sebas? Riddell?
‎[20:05] ‎<‎romangg‎>‎ can I go first? have to leave early for lunch.
‎[20:05] ‎<‎Sho_‎>‎ sure
‎[20:05] ‎<‎Sho_‎>‎ i'll assume bshah is here as well given the above
‎[20:05] ‎<‎Sho_‎>‎ romangg starts so he can lunch, because he's mean and \
doesn't want to read us ‎[20:05] ‎<‎romangg‎>‎ i read it later!! i \
promise!!! ‎[20:05] ‎<‎romangg‎>‎ Pushed Night Color / LibColorCorrect \
patches to KWin, plasma-workspace and plasma-desktop. Testers welcome :) ‎[20:05] \
‎<‎romangg‎>‎ (on wayland) ‎[20:06] ‎<‎Sho_‎>‎ of course (/me hides \
a tear anyway) ‎[20:06] ‎<‎romangg‎>‎ There is currently still a discussion \
about if the kcm should be shown on X or hidden. It currently looks like this on X: \
https://imgur.com/a/sYBdQ ‎[20:06] ‎<‎romangg‎>‎ I'm for showing it because \
(a) users know this way that this functionality exists in general (b) we won't get \
hit with dozens of bug reports asking where they can activate this functionality they \
heared about on omgubuntu / phoronix. (c) gives user small push to try out wayland \
‎[20:06] ‎<‎romangg‎>‎ but we could also hide it on X with a flag in \
desktop file. ‎[20:06] ‎<‎Sho_‎>‎ tricky, I could also imagine it causing \
user frustration ‎[20:07] ‎<‎d_ed‎>‎ I don't see how
‎[20:07] ‎<‎d_ed‎>‎ the only people who have read it, will be KDE fanboys \
who will have also read the line about wayland ‎[20:07] ‎<‎romangg‎>‎ d_ed: \
you're responding to Sho_ or to (b)? ‎[20:07] ‎<‎d_ed‎>‎ b
‎[20:08] ‎<‎Sho_‎>‎ In general I'm a fan of not hiding functionality but \
showing unavailable options and what you need to do for it to become available \
‎[20:08] ‎<‎romangg‎>‎ well, you for sure hold our users in high regard... \
‎[20:08] ‎<‎notmart‎>‎ i think wayland specific stuff shouldn't even be \
visible on x, as i imagine there may be more and more in the future ‎[20:08] \
‎<‎notmart‎>‎ giving an half-greyed out systemsettings ‎[20:08] \
‎<‎Sho_‎>‎ but this will cause some pissy people who say "why do you make \
features wayland exclusive when wayland is still unusable" ‎[20:09] \
‎<‎romangg‎>‎ on X they can still use Redshift of course. ‎[20:09] \
‎<‎romangg‎>‎ but I see that people want to hide it. so I'll look into adding \
this flag ‎[20:09] ‎<‎romangg‎>‎ ...after lunch
‎[20:09] ‎<‎d_ed‎>‎ romangg: X-KDE-OnlyShowOnQtPlatforms=wayland
‎[20:09] ‎<‎romangg‎>‎ d_ed: thanks
‎[20:09] ‎<‎Sho_‎>‎ I'm undecided, I see arguments for both, so I'm fine \
with whatever you decide ‎[20:09] ‎<‎romangg‎>‎ I'm also looking into \
kdesrc-build at the moment to figure out if a cross compile would be possible \
‎[20:10] ‎<‎Sho_‎>‎ if you decide wrong, I can later say I saw it coming, \
muahahahaha ‎[20:10] ‎<‎notmart‎>‎ features will be wayland-exclusive \
anyways ‎[20:10] ‎<‎romangg‎>‎ nice. the politician :P
‎[20:10] ‎<‎romangg‎>‎ I also want to push for T5653 in the next weeks.
‎[20:10] ‎<‎sKreamer‎>‎ Task 5653 "[kwin] Screen recording in Wayland \
session" [Open,High] {KWin,Plasma on Wayland} https://phabricator.kde.org/T5653 \
‎[20:10] ‎<‎notmart‎>‎ (and in general, every new kwin feature will \
probably be so) ‎[20:10] ‎<‎Sho_‎>‎ I was at a United Nations meeting today
‎[20:10] ‎<‎romangg‎>‎ would be great if we can have it in 5.12
‎[20:10] ‎<‎d_ed‎>‎ romangg: good luck
‎[20:10] ‎<‎Sho_‎>‎ forgive me for being politicky
‎[20:11] ‎<‎romangg‎>‎ ok, next
‎[20:11] ‎<‎Sho_‎>‎ next is d_ed
‎[20:11] ‎<‎d_ed‎>‎  - Working on some startup improvements
‎[20:11] ‎<‎d_ed‎>‎  - With aleix, we fixed a deadlock in ksmserver which \
has a huge impact on everyone ‎[20:11] ‎<‎d_ed‎>‎  - Then lots of minor \
things related to that ‎[20:11] ‎<‎d_ed‎>‎  - I need some reviews on me \
removing multihead ‎[20:12] --> Son_Goku (~King_InuY@fedora/ngompa) has joined this \
channel. ‎[20:12] ‎<‎Sho_‎>‎ finally, all multihead bugs gone
‎[20:12] ‎<‎notmart‎>‎ hail hydra
‎[20:12] ‎<‎Sho_‎>‎ Phoronix: Plasma removes support for multiple monitors
‎[20:12] ‎<‎bshah‎>‎ heh
‎[20:12] ‎<‎Sho_‎>‎ you done d_ed?
‎[20:12] ‎<‎d_ed‎>‎ I wrote a line in the bottom of every review saying \
multihead != multiscreen to expect that ‎[20:12] ‎<‎d_ed‎>‎ yep
‎[20:12] ‎<‎Sho_‎>‎ then notmart is next
‎[20:12] ‎<‎notmart‎>‎ ok
‎[20:12] <-- xamindar_ (~quassel@71-15-99-150.dhcp.ftwo.tx.charter.com) has left \
this server (Read error: Connection reset by peer). ‎[20:13] ‎<‎notmart‎>‎ \
hm, expect skreamer to go crazy... ‎[20:13] ‎<‎notmart‎>‎ * almost kill the \
packageurlinterceptor usen in plasma, seems speed up things a bit: \
https://phabricator.kde.org/D9124 ‎[20:13] ‎<‎sKreamer‎>‎ Diff 9124 "make \
packageurlinterceptor as noop as possible" [Needs Review] \
https://phabricator.kde.org/D9124 ‎[20:13] ‎<‎notmart‎>‎ * kpackage loading \
binary rcc files: performance increase looks promising, patches all over the place, \
most pushed, still to go: ‎[20:13] ‎<‎notmart‎>‎ ** \
https://phabricator.kde.org/D9197 ‎[20:13] ‎<‎sKreamer‎>‎ Diff 9197 \
"experimental support for rcc files loading in kpackage" [Needs Revision] \
https://phabricator.kde.org/D9197 ‎[20:13] ‎<‎notmart‎>‎ ** \
https://phabricator.kde.org/D9237 ‎[20:13] ‎<‎sKreamer‎>‎ Diff 9237 "port \
wallpaper config fialog to Package::fileUrl" [Needs Review] \
https://phabricator.kde.org/D9237 ‎[20:13] ‎<‎notmart‎>‎ * global menu: \
make it depening on available visualizations (plasmoid or kwin button) not internal \
settings: ‎[20:13] ‎<‎notmart‎>‎ ** https://phabricator.kde.org/D9215
‎[20:13] ‎<‎sKreamer‎>‎ Diff 9215 "Make Appmenu work based on the presence \
of a visual representation" [Needs Review] https://phabricator.kde.org/D9215 \
‎[20:13] ‎<‎notmart‎>‎ ** https://phabricator.kde.org/D9216 ‎[20:13] \
‎<‎sKreamer‎>‎ Diff 9216 "export org.kde.appmenuview" [Needs Review] \
https://phabricator.kde.org/D9216 ‎[20:13] ‎<‎notmart‎>‎ ** \
https://phabricator.kde.org/D9267 ‎[20:13] ‎<‎sKreamer‎>‎ Diff 9267 "remove \
menubar settings" [Needs Review] https://phabricator.kde.org/D9267 ‎[20:13] \
‎<‎notmart‎>‎ * starting to look into adding autostart of apps support in \
look and feel, for latte-dock (or plank, or conky, or whatever sillyness users want) \
‎[20:13] ‎<‎notmart‎>‎ shut up :) ‎[20:13] ‎<‎notmart‎>‎ \
questions? ‎[20:14] ‎<‎Sho_‎>‎ Phoronix: Plasma removes menu support
‎[20:14] ‎<‎Sho_‎>‎ bshah is next
‎[20:14] ‎<‎bshah‎>‎ okay
‎[20:14] ‎<‎bshah‎>‎ - Fixed plasma mobile images
‎[20:14] ‎<‎bshah‎>‎ - Some bugs which I need help with
‎[20:14] ‎<‎bshah‎>‎     - huge scrollbars
‎[20:14] ‎<‎bshah‎>‎     - broken struts
‎[20:15] ‎<‎bshah‎>‎ - working on packaging multimedia bits will have next \
image with it ‎[20:15] ‎<‎bshah‎>‎ - end
‎[20:15] ‎<‎Sho_‎>‎ Phoronix: Plasma adopts huge scrollbars
‎[20:15] ‎<‎Sho_‎>‎ I'll close then
‎[20:15] ‎<‎Sho_‎>‎ * Wrote a patch for Plasma::Svg not to tear down the \
renderer and repaint when setImagePath is called with the same arg again, speeds up \
both plasmashell startup (0.5%) and Simple Menu opening ‎[20:15] ‎<‎Sho_‎>‎ \
* Wrote another patch for Plasma::Svg to cache the scale factor to initialize with in \
a static member to work being done twice on systems with non-1x scaling, speeds up \
plasmashell startup by 2% ‎[20:15] ‎<‎Sho_‎>‎ * Spent some more time \
banging on Plasma::Svg, chased down some rabbit holes, didn't find anything too \
promising ‎[20:15] ‎<‎Sho_‎>‎ * Implemented QQmlParserStatus in \
FolderModel, saves some churn during initialization ‎[20:15] ‎<‎Sho_‎>‎ * \
Wrote a patch to reduce vertical gutter width in Folder View icon views based on user \
feedback, needs more work though ‎[20:15] ‎<‎Sho_‎>‎ * Wrote an \
optimization patch for the panel containment, saving one object per applet ‎[20:15] \
‎<‎Sho_‎>‎ * Wrote a patch for Plasma::FrameSvgItem avoiding an update during \
instanciation before the component is complete ‎[20:15] ‎<‎Sho_‎>‎ * Wrote \
a patch for KNotification to lazy-instanciate & lazy-load plugins, speeds up login \
time as well as mem usage of basically all apps, as well as shortens latency of their \
first notification event - helps with Plasma startup speed via kded notifications \
‎[20:15] ‎<‎Sho_‎>‎ * Moved KGamma init to the right autostart phase to \
speed up login a little bit ‎[20:15] ‎<‎Sho_‎>‎ * Came up with a plan for \
how to move KAccess out of early startup, make more of it work on Wayland and kill \
its permanent process - the bell goes into a KWin effect, the screen reader toggle \
action goes into a kded module, the rest goes into regular autostart ‎[20:15] \
‎<‎Sho_‎>‎ * Minor improvement to existing KAccess KCMInit to save a config \
file read during login ‎[20:15] ‎<‎Sho_‎>‎ * Wrote patch to speed up \
plasmashell startup and save a little memory by not setting a window icon (it's not \
needed due to NET::WM2DesktopFileName) ‎[20:15] ‎<‎Sho_‎>‎ * Resurrected \
David's patch speeding up Plasma::Dialog and saving some memory by not setting a \
window icon (it's not needed due to NET::WM2DesktopFileName) ‎[20:15] \
‎<‎Sho_‎>‎ * Wrote patches to speed up xembedsniproxy, kactivitymanagerd and \
kglobalaccel5 startups by not loading KDE platform module ‎[20:15] \
‎<‎Sho_‎>‎ * Wrote a patch for the Virtual Desktops KCM to make the \
number-of-rows field to have the correct lower bound of 1 instead of 0 (user bug \
report came in during the day) ‎[20:15] ‎<‎Sho_‎>‎ * Ported launch feedback \
KCM rewrite to new base items from Marco's patch, done from my end ‎[20:15] \
‎<‎Sho_‎>‎ * Sped up Simple Menu opening by reducing delegate complexity and \
reworking delegate pre-caching ‎[20:15] ‎<‎Sho_‎>‎ * Reworked Simple Menu \
keyboard navigation: ‎[20:15] ‎<‎Sho_‎>‎ ** Arrow nav after enterign search \
terms works again ‎[20:15] ‎<‎Sho_‎>‎ ** Horizontal arrow nav through \
search results works immediately, no need to arrow down into the grid first \
‎[20:15] ‎<‎Sho_‎>‎ ** Tab/Shift+Tab now work to cycle through the search \
field, the system favorites and the category list ‎[20:15] ‎<‎Sho_‎>‎ * \
Released Simple Menu v1.0.5 and v1.0.6 (performance work + keyboard work + hotfix) \
‎[20:15] ‎<‎Sho_‎>‎ * Worked around a Qt bug causing a scroll indicator \
line to appear at the bottom of Simple Menu pages (which can't be scrolled vertically \
...), fix is in plasma-framework ‎[20:15] ‎<‎Sho_‎>‎ * Ported ksystraycmd \
to KF5 (https://cgit.kde.org/scratch/hein/ksystraycmd.git/) ‎[20:15] \
‎<‎Sho_‎>‎ * Non-Plasma: Improved Dolphin startup speed on systems without \
kio-stash installed (i.e. most) by using a faster way to determine whether it is \
installed or not ‎[20:15] ‎<‎Sho_‎>‎ * Personal: On vacation Dec 18th-Jan \
2nd

And the rest from another client due to a spam flood kill after pasting the above:

notmart (IRC)
20:16yeah, i got this warning by the spam thing as well :/

bshah (IRC)
20:16wut
20:16Fuchs: ^

d_ed (IRC)
20:16he did a kamikaze git log

notmart (IRC)
20:16really looks like irc is on its last legs and need matrix asap...
20:17either you get spammers or anti spam bots that kill you for nothing

Sho_ (IRC)
20:17I got floodkilled
20:18here's my full report including my vacation notice at the end: \
https://paste.kde.org/p1pwrbpzm/knkvfe/raw

pursuivant (IRC)
20:18kde-gtk-config (master) v5.11.2-19-geec9a6c * Jan Grulich: (2 files in 2 \
dirs) 20:18Fix build
20:18When building gtk2 based stuff we need to be sure gtk2 includes are included \
first 20:18and also search properly for gio includes
20:18https://commits.kde.org/kde-gtk-config/eec9a6cd2aec6617d4dc03110248238293d77e97
kde-gtk-config.git - GTK2 and GTK3 Configurator for KDE.
index : kde-gtk-config.git 2.1 2.2 Plasma/5.10 Plasma/5.11 Plasma/5.2 Plasma/5.3 \
Plasma/5.4 Plasma/5.5 Plasma/5.6 Plasma/5.7 Plasma/5.8 Plasma/5.9 frameworks

Sho_ (IRC)
20:18unless there's questions that's /meeting

d_ed (IRC)
20:18Sho_: anything stuck in review?

Sho_ (IRC)
20:18d_ed: the dolphin patch and the kglobalaccel5 one

d_ed (IRC)
20:18ok, I know them

Sho_ (IRC)
20:18everything else is in
20:18which makes me 99% happy



[prev in list] [next in list] [prev in thread] [next in thread]