Add option to repeatedly output status information#633
Add option to repeatedly output status information#633602p wants to merge 7 commits intohirschmann:betafrom
Conversation
I see that my e580 seems to behave in the same way, and I think this is a bug: Author of this config probably wanted to set 80 as hex not as decimal. on my device this is what happens on exit: hex 80 = dec 128 => factory auto mode hex 50 = dec 80 => full speed of fan
This configuration is an exact copy of "Asus ROG G752VT" as it also works here. In case it turns out to be important in the future: this is for a ROG G752VS 7th Gen (since there may also be versions with 6th Gen Intel processors).
|
Looks like I made this PR against |
|
Thank you very much for your contribution! Master is actually meant to be the public integration branch for this repository, so it was perfectly fine to base your patch on master. I'm planning to create a REST API for the service to make it easier to access from other scripts/programs. |
|
hey what about if you add a fan overcloking feature? |
8 participants
This patch introduces a
-poption to thestatusverb of the cli tool that causes the status to be repeatedly output. The rate is configurable as an optional argument to the parameter (measured in seconds, supporting float values.)The rationale behind this patch is that I like to have a display of the current (target) fan speed in my system bar, and the CPU overhead from repeated mono startup was a major contributor to load on an otherwise idle system. Keeping a copy of the cli tool running in the background avoids that. I use this as an i3blocks element by running
nbfc status -p 1 -f 0 | awk '/Target/ { printf("%3d\n", $5); system("") }'withinterval=persist.If this is merged, we should be sure to update the CLI examples page on the Wiki.