starred/nbfc-hirschmann

Fork 0

mirror of https://github.com/hirschmann/nbfc.git synced 2026-04-25 16:45:53 +03:00

[GH-ISSUE #704] Add variable to control temperature polling rate #620

New issue

Closed

opened 2026-02-26 00:33:08 +03:00 by kerem · 4 comments

kerem commented

2026-02-26 00:33:08 +03:00

Owner

Originally created by @Dainslaif on GitHub (Jun 6, 2019).
Original GitHub issue: https://github.com/hirschmann/nbfc/issues/704

Is your feature request related to a problem? Please describe.
The temperature polling rate is hard-set to 3 seconds (per comments at https://github.com/hirschmann/nbfc/issues/703), so it lags quite a bit behind my computer's actual temperature, letting it spike before it kicks in and turns on the fans.

Describe the solution you'd like
Add in a variable that lets the user define their temperature polling rate. This would set how often NBFC updates the temperature value it operates from.

Additional context
Probably should have a low max value (3 seconds?) for the sake of safety.

Originally created by @Dainslaif on GitHub (Jun 6, 2019). Original GitHub issue: https://github.com/hirschmann/nbfc/issues/704 **Is your feature request related to a problem? Please describe.** The temperature polling rate is hard-set to 3 seconds (per comments at https://github.com/hirschmann/nbfc/issues/703), so it lags quite a bit behind my computer's actual temperature, letting it spike before it kicks in and turns on the fans. **Describe the solution you'd like** Add in a variable that lets the user define their temperature polling rate. This would set how often NBFC updates the temperature value it operates from. **Additional context** Probably should have a low max value (3 seconds?) for the sake of safety.

kerem

2026-02-26 00:33:08 +03:00

closed this issue
added the
Stale
label

kerem commented

2026-02-26 00:33:09 +03:00

Author

Owner

@hirschmann commented on GitHub (Jun 6, 2019):

The temperature polling rate is the same as the EC polling rate and can be set via config.
Only the UI update rate is fixed at 3 seconds. The UI update rate doesn't affect the functionality of the NBFC service at all.

@hirschmann commented on GitHub (Jun 6, 2019): The temperature polling rate is the same as the EC polling rate and can be set via config. Only the UI update rate is fixed at 3 seconds. The UI update rate doesn't affect the functionality of the NBFC service at all.

kerem commented

2026-02-26 00:33:09 +03:00

Author

Owner

@Dainslaif commented on GitHub (Jun 6, 2019):

Interesting, is there a reason they don't just both update at the same rate?

@Dainslaif commented on GitHub (Jun 6, 2019): Interesting, is there a reason they don't just both update at the same rate?

kerem commented

2026-02-26 00:33:09 +03:00

Author

Owner

@hirschmann commented on GitHub (Jun 6, 2019):

I wanted to avoid putting too much load on the system, especially when multiple clients are running at the same time. Also the client currently has no way of querying the service's poll interval.

I consider pushing changes to the client instead of polling in the future, though.

@hirschmann commented on GitHub (Jun 6, 2019): I wanted to avoid putting too much load on the system, especially when multiple clients are running at the same time. Also the client currently has no way of querying the service's poll interval. I consider pushing changes to the client instead of polling in the future, though.

kerem commented

2026-02-26 00:33:09 +03:00

Author

Owner

@github-actions[bot] commented on GitHub (Dec 5, 2019):

This issue is stale because it has been open more than 180 days with no activity. If nobody comments within 7 days, this issue will be closed

@github-actions[bot] commented on GitHub (Dec 5, 2019): This issue is stale because it has been open more than 180 days with no activity. If nobody comments within 7 days, this issue will be closed

kerem referenced this issue

2026-02-26 00:35:05 +03:00

[PR #620] [MERGED] Asus Zenbook UX330UA #1321

kerem referenced this issue

2026-02-26 00:35:07 +03:00

[PR #660] [CLOSED] 1 #1329

kerem referenced this issue

2026-02-26 00:35:10 +03:00

[PR #717] [CLOSED] Lenovo Yoga C930 #1343

kerem referenced this issue

2026-02-26 00:35:11 +03:00

[PR #763] [CLOSED] Addition request on the software #1351