[PR #633] Add option to repeatedly output status information #1322

Open
opened 2026-02-26 00:35:06 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/hirschmann/nbfc/pull/633
Author: @602p
Created: 2/5/2019
Status: 🔄 Open

Base: betaHead: master


📝 Commits (7)

📊 Changes

8 files changed (+516 additions, -6 deletions)

View changed files

Configs/Acer Predator G3-572.xml (+242 -0)
Configs/Asus ROG G752VS.xml (+112 -0)
Configs/HP Laptop 14-cm0xxx.xml (+67 -0)
📝 Configs/HP ProBook 4530s.xml (+29 -4)
📝 Configs/Lenovo ThinkPad 13.xml (+2 -2)
Configs/Lenovo ThinkPad x220i.xml (+46 -0)
📝 Core/NbfcCli/CommandLineOptions/StatusVerb.cs (+9 -0)
📝 Core/NbfcCli/Program.cs (+9 -0)

📄 Description

This patch introduces a -p option to the status verb 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("") }' with interval=persist.

If this is merged, we should be sure to update the CLI examples page on the Wiki.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/hirschmann/nbfc/pull/633 **Author:** [@602p](https://github.com/602p) **Created:** 2/5/2019 **Status:** 🔄 Open **Base:** `beta` ← **Head:** `master` --- ### 📝 Commits (7) - [`4a3d004`](https://github.com/hirschmann/nbfc/commit/4a3d0041687edea5bc96717570c2a4e0f9e24453) Add Lenovo ThinkPad x220i config (#588) - [`79d36ed`](https://github.com/hirschmann/nbfc/commit/79d36ed914bb0440ac7cede39447ca9075ec2a15) Add Acer Predator G3-572 config (#586) - [`e4299e3`](https://github.com/hirschmann/nbfc/commit/e4299e3cb45d6174015368f2832bc08c25e76070) Fix reset value in Lenovo ThinkPad 13 config (#604) - [`8b19604`](https://github.com/hirschmann/nbfc/commit/8b19604116d650a7125817017f37751f2ce60f39) Add Asus ROG G752VS configuration (#610) - [`4354594`](https://github.com/hirschmann/nbfc/commit/4354594f65a756aee458ef4982c40de427225454) Add HP Laptop 14-cm0xxx config (#615) - [`c3dd9dc`](https://github.com/hirschmann/nbfc/commit/c3dd9dc6f94f2382428a65009c0f750cf0bc64f0) Improve config for HP ProBook 4530s (#618) - [`f83ac29`](https://github.com/hirschmann/nbfc/commit/f83ac2940d0f910b66e741557f595474ce662546) Add option to repeatedly output status information ### 📊 Changes **8 files changed** (+516 additions, -6 deletions) <details> <summary>View changed files</summary> ➕ `Configs/Acer Predator G3-572.xml` (+242 -0) ➕ `Configs/Asus ROG G752VS.xml` (+112 -0) ➕ `Configs/HP Laptop 14-cm0xxx.xml` (+67 -0) 📝 `Configs/HP ProBook 4530s.xml` (+29 -4) 📝 `Configs/Lenovo ThinkPad 13.xml` (+2 -2) ➕ `Configs/Lenovo ThinkPad x220i.xml` (+46 -0) 📝 `Core/NbfcCli/CommandLineOptions/StatusVerb.cs` (+9 -0) 📝 `Core/NbfcCli/Program.cs` (+9 -0) </details> ### 📄 Description This patch introduces a `-p` option to the `status` verb 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("") }'` with `interval=persist`. If this is merged, we should be sure to update the CLI examples page on the Wiki. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nbfc-hirschmann#1322
No description provided.