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

Visualizer updates #60

Merged
merged 1 commit into from
Mar 14, 2023
Merged

Visualizer updates #60

merged 1 commit into from
Mar 14, 2023

Conversation

janaknat
Copy link
Contributor

Updates:

  • CPU Utilization graph y-axis is always 100%.
  • Diskstat read, sector graph updates.
  • VMStat graph updates.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@@ -2,3 +2,5 @@ pub static KB_FACTOR: u64 = 1024;
pub static MB_FACTOR: u64 = 1048576;
pub static TIME_S_FACTOR: u64 = 1000;
pub static FACTOR_OF_ONE: u64 = 1;
pub static MULT_FACTOR: u64 = 1;
pub static SECTORS: u64 = 512;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That name is pretty vague.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SECTORS?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Darnit, I know why two lines got selected ..... yes I mean SECTORS

Comment on lines +228 to +230
if unit == "MB" {
factor = MB_FACTOR;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unit "MB" only occurs for keys that contain "Sectors"???

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

Updates:
* CPU Utilization graph y-axis is always 100%.
* Diskstat read, sector graph updates.
* VMStat graph updates.
@janaknat janaknat force-pushed the visualizer_updates branch from 05748c5 to 1b9e5ab Compare March 14, 2023 16:16
@@ -2,3 +2,5 @@ pub static KB_FACTOR: u64 = 1024;
pub static MB_FACTOR: u64 = 1048576;
pub static TIME_S_FACTOR: u64 = 1000;
pub static FACTOR_OF_ONE: u64 = 1;
pub static MULT_FACTOR_OF_ONE: u64 = 1;
pub static MULT_SECTORS: u64 = 512;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That name is still pretty unhelpful

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:(

@janaknat janaknat merged commit c4d488e into main Mar 14, 2023
@janaknat janaknat deleted the visualizer_updates branch July 7, 2023 20:12
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

Successfully merging this pull request may close these issues.

2 participants