Skip to content

Set a better redraw time to avoid High CPU Usage#99

Open
rimolive wants to merge 1 commit into
MrRio:masterfrom
rimolive:master
Open

Set a better redraw time to avoid High CPU Usage#99
rimolive wants to merge 1 commit into
MrRio:masterfrom
rimolive:master

Conversation

@rimolive

@rimolive rimolive commented Jul 9, 2017

Copy link
Copy Markdown

Fix issues #82 and #55

Note: This fix just changed a hard-coded value. Need to discuss if
this value shouldn't be parametrized by the user via command-line

Fix issues #82 and #55

Note: This fix just changed a hard-coded value. Need to discuss if
this value shouldn't be parametrized by the user via command-line
@1mw

1mw commented Jul 26, 2017

Copy link
Copy Markdown

I personally believe this should be up to the user, but have no problem with simply changing the default to a more reasonable value. Does vtop have existing logic to load configuration files?

@rimolive

Copy link
Copy Markdown
Author

It's possible to make it configurable, but I don't see a reason to collect information every 100ms or 5 minutes. 1 second is enough, but if needed I can change to make it configurable.

@1mw

1mw commented Aug 1, 2017

Copy link
Copy Markdown

I'm definitely in favor of configurability (I would love to be able to customize this), but I suppose that change could come later if necessary.

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