Skip to content

Add I/O read per second, fix bug sizing of io overview#61

Merged
navidpadid merged 2 commits into
mainfrom
read-bytes-add
Apr 16, 2026
Merged

Add I/O read per second, fix bug sizing of io overview#61
navidpadid merged 2 commits into
mainfrom
read-bytes-add

Conversation

@navidpadid
Copy link
Copy Markdown
Owner

This pull request extends the process overview functionality to support and display read byte rates (RD/s) alongside existing metrics, and updates the codebase to handle size fields with units (e.g., "1 GB", "100 MB") for I/O statistics. It also adds comprehensive test coverage for these new features.

Feature enhancements

  • Added support for parsing and displaying read_bytes statistics with unit suffixes (B, KB, MB, GB, TB), similar to write_bytes, in the process overview plots. This includes a new RD/s sparkline and label in the UI.

Parsing improvements

  • Introduced the parse_size_field_bytes function to robustly handle size fields with units when parsing I/O statistics. This ensures correct interpretation of values such as "1 GB" or "512 MB".

Testing updates

  • Updated and expanded tests to cover new parsing logic and ensure the overview output includes the new RD/s metric, as well as correct handling and display of unit-suffixed values.

@navidpadid navidpadid merged commit 6751dbb into main Apr 16, 2026
5 checks passed
@navidpadid navidpadid deleted the read-bytes-add branch April 16, 2026 01:27
@navidpadid navidpadid restored the read-bytes-add branch April 16, 2026 01:32
@navidpadid navidpadid deleted the read-bytes-add branch April 16, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant