Review Request 104015: A usable system monitor plasmoid
rhn rhn
kdde.rhn at porcupinefactory.org
Tue Jul 29 09:57:24 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/104015/
-----------------------------------------------------------
(Updated July 29, 2014, 9:57 a.m.)
Status
------
This change has been discarded.
Review request for Plasma.
Repository: kdeplasma-addons
Description
-------
A plasmoid intended to be used on a panel. Inspired by gnome-multiload-applet and gkrellm.
Before I decided to write this plasmoid, I considered other monitors. Here's what I found:
- system-monitor:
* not pixel-perfect: it uses antialiasing and Bezier curves
* in stacking mode, it adds a few pixels to all values, so a 0 network throughput still shows 2 belts.
* impossible to read at a glance due to low contrast and no custom colors
* shows only basic properties
* graphs change features depending on size
* deal-breaker: impossible to set custom height or width
- system-load-viewer: almost there, but no history display
- The Only Right RAM Monitor: too slow and RAM only
This plasmoid attempts to solve all those problems while introducing new ones :)
What's fixed in this one:
- many graphs in one applet
- graphs laid out on a grid
- forces graph width or height (depending on mode)
- can force row or column number (depending on mode)
- arbitrary colors for all graph types
- pixel-perfect
- handles runtime CPU/network addition/removal (CPU untested)
Other stuff:
- labels only for network interfaces
- resizable labels
- network transmit graph stacks on top
- C++, a monitor should measure resources, not use them
- no way to disable graphs :)
- no way to reorder graphs
- network interfaces are not sorted
- all CPUs are displayed
- can't change colors for a distinct graph
- will not fit in container if the wrong layout params are selected (forcing a dimension to be more than available)
- the original idea was to synchronize all readings, but it seems impossible with plasma data engines
It's my first project in Qt, so the code will not be as nice as it could be. I rewrote the plotter and the layout, so those are the most likely to contain errors/memory leaks. Also, I don't know which of the above problems need to be solved and which decisions are acceptable, but I'll happily apply all fixes to make this accepted.
Diffs
-----
applets/CMakeLists.txt 1e1ef75
applets/graphmon/CMakeLists.txt PRE-CREATION
applets/graphmon/color-config.ui PRE-CREATION
applets/graphmon/cpuload.h PRE-CREATION
applets/graphmon/cpuload.cpp PRE-CREATION
applets/graphmon/general-config.ui PRE-CREATION
applets/graphmon/graphmon.h PRE-CREATION
applets/graphmon/graphmon.cpp PRE-CREATION
applets/graphmon/netthroughput.h PRE-CREATION
applets/graphmon/netthroughput.cpp PRE-CREATION
applets/graphmon/plasma-applet-graphmon.desktop PRE-CREATION
applets/graphmon/ramusage.h PRE-CREATION
applets/graphmon/ramusage.cpp PRE-CREATION
applets/graphmon/rigidgridlayout.h PRE-CREATION
applets/graphmon/rigidgridlayout.cpp PRE-CREATION
applets/graphmon/simpleplotter.h PRE-CREATION
applets/graphmon/simpleplotter.cpp PRE-CREATION
applets/graphmon/swapusage.h PRE-CREATION
applets/graphmon/swapusage.cpp PRE-CREATION
Diff: https://git.reviewboard.kde.org/r/104015/diff/
Testing
-------
Display, layout, configuration, persistence.
Screenshots
-----------
Graph on panel and general config section
https://git.reviewboard.kde.org/r/104015/s/437/
Thanks,
rhn rhn
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140729/1cbf0689/attachment.html>
More information about the Plasma-devel
mailing list