Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Congestion graph: congestion metrics hard to read #87

Open
ptrd opened this issue Dec 22, 2024 · 0 comments
Open

Congestion graph: congestion metrics hard to read #87

ptrd opened this issue Dec 22, 2024 · 0 comments

Comments

@ptrd
Copy link

ptrd commented Dec 22, 2024

Describe the bug
In the graph shown in the tab named "Congestion", ironically the congestion metrics are quite hard to read, because of the following:

  • the legend (explaining that purple is the congestion window and olive is bytes in flight) is obscuring the scale and the last section of the congestion graph. In my experience this is always the case, apparently does not depend on the actual graph. Kind of a shame, because a above the legend there is plenty of whitespace to show the legend without hiding anything.
  • the values on the scale do not correspond to the grid lines. I understand the grid lines correspond with the #bytes scale on the left, so this could be seen as "intentional". However, the scale at the right seems to be arbitrarily chosen, given the fact that the largest value of the congestion window usually is less the a third of the total scale. If the values for the ticks would be selected in a way that the values at the left are a multiple of the values at the right, the grid lines could serve both scales at the same time!
  • it would be nice it the actual values of the congestion-window line could be shown when hovering over it, just like it does with the blue line.

To Reproduce

  • Pick any qlog file produced by the interop transfer test case
  • Go to the tab "Congestion"

Expected behavior

  • The legend should not hide part of the line or scale
  • There should be grid lines attached to the scale at the right

Screenshots
See attachments
Screenshot 2024-12-22 at 21 27 14
Screenshot 2024-12-22 at 21 27 23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant