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

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

--- Comment #4 from Albert Rafetseder <albert.rafetseder+wireshark@xxxxxxxxxxxx> 2011-08-23 02:53:53 PDT ---
(In reply to comment #3)
> 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.

Well, thanks for considering our patch in the first place. Should we rather
make it an "Advanced..." option for the y axis? That would leave the original
appearance of the IO Graph window intact, and we can address the naming issue
(see below) as well.

> 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.

Granted. I'd like to add it shouldn't be called "filter" to avoid confusion
with display filters of the "tcp.flags.syn==1" kind.

> 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.

Oops, we'll fix that and post another patch.

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