Review Request 129917: Add a cache monitor to the System Load Viewer applet

Sebastian Kügler sebas at kde.org
Tue Feb 7 09:13:05 GMT 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129917/#review102439
-----------------------------------------------------------




applets/systemloadviewer/package/contents/config/main.xml (line 52)
<https://git.reviewboard.kde.org/r/129917/#comment68316>

    I think it's overkill to enable this by default, given it's only interesting to a limited group of users, and others probably won't understand it, or its utility.


- Sebastian Kügler


On Feb. 6, 2017, 8:05 p.m., Pascal VITOUX wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129917/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2017, 8:05 p.m.)
> 
> 
> Review request for kde-workspace and Plasma.
> 
> 
> Repository: kdeplasma-addons
> 
> 
> Description
> -------
> 
> Add a cache monitor to the System Load Viewer applet using 'cache/dirty' and 'cache/writeback' infos from ksysguard 5.9.0
> 
> The monitor displays the dirty + writeback amount proportionaly scaled with the maximal amount reached, until the remaining dirty amount goes below a minimal threshold of 10MB.
> 
> An interesting usecase is to show the real progress of a datas transfer to a slow storage device like a USB drive.
> 
> 
> Diffs
> -----
> 
>   applets/systemloadviewer/package/contents/config/main.xml 6bf16d5aa 
>   applets/systemloadviewer/package/contents/ui/ColorSettings.qml b9247aa15 
>   applets/systemloadviewer/package/contents/ui/GeneralSettings.qml f1ab40a1b 
>   applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 5a0bc0649 
> 
> Diff: https://git.reviewboard.kde.org/r/129917/diff/
> 
> 
> Testing
> -------
> 
> I use it since several months without issue.
> 
> 
> File Attachments
> ----------------
> 
> panel applets : circular, bar and compact bar monitors
>   https://git.reviewboard.kde.org/media/uploaded/files/2017/02/06/9067d5be-c1b4-4168-be20-76965b4a40ac__panel.png
> desktop applet, with tooltip
>   https://git.reviewboard.kde.org/media/uploaded/files/2017/02/06/4eb7a092-7368-448f-8385-7c18ab298389__desktop-tooltip.png
> desktop applet, circular monitors
>   https://git.reviewboard.kde.org/media/uploaded/files/2017/02/06/26f769fb-85e9-45f7-90e6-55035770be6e__desktop-circular.png
> 
> 
> Thanks,
> 
> Pascal VITOUX
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20170207/616ffde2/attachment.htm>


More information about the kde-core-devel mailing list