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