Network notification returns wrong results

Bug #869282 reported by zorun
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
byobu
Fix Released
High
Unassigned
byobu (Ubuntu)
Fix Released
High
Unassigned

Bug Description

The notification that shows up/down network throughput calculates it wrong.

The factor is roughly 0.75: when having a 120 KB/s throughput, byobu shows 80 KB/s.
I've checked the "real" value with three independant tools (iftop, nethogs, and conky) and they all agree.

This bug was introduced quite recently, when the system that handles notifications "ticks" has changed.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Hmm, upload, download, or both?

It just tested upload very thoroughly, and it seems to be dead on. I saturated a 2Mbps upload connection and that's exactly what it was reading.

Changed in byobu:
importance: Undecided → High
status: New → Incomplete
Changed in byobu (Ubuntu):
importance: Undecided → High
status: New → Incomplete
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Actually, Scott committed a change recently:

  [Scott Moser]
  * usr/lib/byobu/.shutil: fix rounding error in fpdiv

This might actually be the problem. Could you retest, again, with 4.43 when I release?

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Here's a couple of screen shots showing it working pretty well here for me...

Revision history for this message
Dustin Kirkland  (kirkland) wrote :
Revision history for this message
Dustin Kirkland  (kirkland) wrote :
Changed in byobu:
status: Incomplete → Fix Committed
Changed in byobu (Ubuntu):
status: Incomplete → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package byobu - 4.43-0ubuntu1

---------------
byobu (4.43-0ubuntu1) precise; urgency=low

  [ Dustin Kirkland ]
  * usr/bin/byobu, usr/lib/byobu/ip_address, usr/share/man/man1/byobu.1:
    - support IP_EXTERNAL fetching, LP: #873736
    - put ip address in the terminal title bar, since we ran out of room
      for it in tmux
  * usr/bin/byobu-select-session: LP: #869390
    - fix select session to obey BYOBU_BACKEND
  * configure.ac, usr/bin/Makefile.am: LP: #869377
    - provide byobu-tmux and byobu-screen symlinks
  * configure.ac, README: LP: #861291
    - configure should not change files under vcs control
    - compiling users need to ensure that BYOBU_PREFIX is set in
      their environment
  * usr/bin/byobu-launcher-install, usr/bin/byobu-launcher-uninstall:
    - LP: #861534
    - ensure launcher command removed from each file before installing
  * usr/lib/byobu/.dirs:
    - use /dev/shm for cache, LP: #872551, #869279
    - search and use a shared dir when possible
  * usr/lib/byobu/battery: LP: #851100
    - rework the battery logic to use /sys instead of /proc
  * usr/share/man/man1/byobu.1: LP: #713879
    - note too many files open ulimit error in documentation

  [ Daniel Hahler ]
  * usr/bin/byobu, usr/bin/byobu-launcher-install, usr/bin/byobu-
    launcher-uninstall: Use BYOBU_PREFIX explicitly, LP: #861302
  * debian/rules, README: LP: #854868
    - refactor autoconf into debian/rules

  [Scott Moser]
  * usr/lib/byobu/.shutil: fix rounding error in fpdiv, LP: #869282
 -- Dustin Kirkland <email address hidden> Fri, 21 Oct 2011 13:43:57 -0500

Changed in byobu (Ubuntu):
status: Fix Committed → Fix Released
Changed in byobu:
status: Fix Committed → Fix Released
Revision history for this message
Martin Vala (vala-martin) wrote :

With version byobu-5.21 i have similar problem. Check screenshot

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.