From: Hannu Nyman Date: Tue, 2 Feb 2016 12:22:16 +0000 (+0200) Subject: luci-app-statistics: Remove usually empty std.dev. graph from ping page X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=a23d69eb711df1d875994c225b43f3cf193ca139;p=project%2Fluci.git luci-app-statistics: Remove usually empty std.dev. graph from ping page Remove the usually empty standard deviation graph from the ping page. The graph is empty for most users as collectd measures standard deviation of individual pings inside the general interval of statistics collection. Default setting for both ping interval and general collection interval is 30s in Luci statistics, meaning just 1 ping per interval, which leads to empty graph. (To provide relveant data, the ping interval should be 1/4-1/5 of the general collection interval. Even then the graph does not look very informative due to different scaling than the latency graph.) Note that this commit does not change collectd itself, which continues to collect and calculate also the std.dev. data, which can be fetched with 'rrdtool' if needed. Signed-off-by: Hannu Nyman (cherry-picked from 8f8957f90f2c593e233d867cab48519a2215b8a0) --- diff --git a/applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua b/applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua index f0a5948169..6ecdb5c8ad 100644 --- a/applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua +++ b/applications/luci-app-statistics/luasrc/statistics/rrdtool/definitions/ping.lua @@ -23,16 +23,6 @@ function rrdargs( graph, plugin, plugin_instance, dtype ) types = { "ping_droprate" }, options = { ping_droprate = { noarea = true, overlay = true, title = "%di" } } - } }, - - -- Ping standard deviation - { title = "%H: ICMP Standard Deviation", - vlabel = "ms", - number_format = "%5.2lf ms", - data = { - types = { "ping_stddev" }, - options = { ping_stddev = { - noarea = true, overlay = true, title = "%di" } } - } }, + } } } end