Wireshark-bugs: [Wireshark-bugs] [Bug 6236] Calculate moving averages in IO Graphs

Date: Tue, 23 Aug 2011 08:14:07 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6236

--- Comment #8 from Akos Lukovics <akos321@xxxxxxxxx> 2011-08-23 08:14:07 PDT ---
(In reply to comment #3)

I have attached a new diff with a few changes.

> I don't really like the placement of the filter combobox in the GUI.  We don't
> have the standard 6 pixel border in the IO Graph because we want the window to
> be as small as possible, and the filter expands the window size.  And we get a
> large unused area below the Graph controls which does not look nice.
>

The feature is now integrated in the Graphs box instead of the Y Axis box, so
there should not be any extra unused space. Still, it makes the IO Graph window
a bit wider.

> We only have one filter, called "M.avg", which does not tell me anything what
> it does.  We need some clarification in the GUI, and this feature needs to be
> documented in the user guide.
>

The name is now changed to "Smooth" and "Moving avg, XX samples". The feature
is not yet documented, as the feature should be finalized first.

> It seems like a bug in the calculation of the leftmost values.  Try having a
> large capture (or set a small tick interval) to get a scrollbar.  When
> scrolling the graph the graph changes, which makes it non reliable.  The
> filtered data has to be calculated for the whole data set, not only the
> displayed ones.

The bug with the scrolling should be fixed now.

Also, the algorithm has been changed to Central Moving Average, which means not
only past but also future values (if existant) are taken into account. This
makes the smoothing somewhat better on the first tapped intervals.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.